dlmarion commented on PR #48:
URL: 
https://github.com/apache/accumulo-classloaders/pull/48#issuecomment-3772840915

   > This is still making a global change to the VM which can make for fun 
debugging (have debugged code X that was broken by some odd global change that 
unrelated code Y did multiple times).
   
   I think this could be resolved by not using the ServiceLoader and instead 
constructing URL objects using the constructor [URL(String context, String 
spec, URLStreamHandler 
hander)](https://github.com/openjdk/jdk/blob/jdk-17%2B35/src/java.base/share/classes/java/net/URL.java#L594).
 Looking at the 
[constructor](https://github.com/openjdk/jdk/blob/jdk-17%2B35/src/java.base/share/classes/java/net/URL.java#L515)
 that just takes a String for the URL, we would pass `null` for the context 
parameter.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to