NianticRyan commented on issue #24098:
URL: https://github.com/apache/beam/issues/24098#issuecomment-1338022001

   I have a bit more context on the issue now. 
   
   Prior to commit #16275, we were just calling `getClasspathFiles()` on the 
Classgraph instance which generates the `ScanResult` internally within the 
ClassGraph library. Since they are generating the report internally, they are 
closing it. 
   However, now we are calling `scan(1)` which returns a `ScanResult`. Since we 
are now holding the handle to `ScanResult`, we need to close it.
   
   So yeah, either rolling back to the previous change and upgrading to the 
latest ClassGraph or making a change such as I or @kennknowles suggested.
   
   If possible, I'd like someone else to submit the PR. I've been trying to 
submit a PR over the past couple of weeks and github's automated systems have 
suspended my account twice so far for reasons that they won't tell me. Every 
time they suspend my account, it also hides this bug report.


-- 
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