Require Java 25 #7305
Require Java 25 #7305
Conversation
|
@labkey-jeckels there are a couple other places that should be updated ( |
Updated. I also eliminated |
labkey-susanh
left a comment
labkey-susanh
left a comment
There was a problem hiding this comment.
I think I'm not fully understanding how the ConnectionState and Cleaner work since JavaDocs say the state should have no reference to the object being cleaned, but then I'm not sure how it can be used to clean anything.
In any case, I think the || true hacks want removing.
| if (MiniProfiler.isCollectTroubleshootingStackTraces()) | ||
| String url = null; | ||
| String threadName = null; | ||
| if (MiniProfiler.isCollectTroubleshootingStackTraces() || true) // TODO - remove hack! |
| private final StackTraceElement[] _debugCreated; | ||
| private final @Nullable DbScope _scope; | ||
| private final @Nullable Connection _connection; | ||
| private final ResultSet _rs; // This is the underlying result set |
There was a problem hiding this comment.
Is this OK? I thought the state was not supposed to have a reference to the object being cleaned?
| public static StackTraceElement[] getTroubleshootingStackTrace() | ||
| { | ||
| if (isCollectTroubleshootingStackTraces()) | ||
| if (isCollectTroubleshootingStackTraces() || true) // TODO - remove hack |
Rationale
We're upgrading to require Java 25.
Changes
Tasks 📍