Aklakan commented on issue #3755:
URL: https://github.com/apache/jena/issues/3755#issuecomment-3904779359

   The thing is, that QueryIterPlainWrapper wraps a plain iterator. Plain 
iterators may have close() but they don't have abort. One solution could be to 
add abort to atlas iterators: Move the abort logic from QueryIteratorBase to an 
atlas AbortableIterator<T> and have both atlas Iterators and the revised 
QueryIteratorBase inherit the same mechanism.
   
   Normally, the QueryIterPlainWrapper would honor abort on the next call to 
`hasNextBinding / moveToNextBinding` - so the close might not be needed.
   But I am worried that removing the forced close from 
`QueryIterPlainWrapper.closeIterator` might cause blocking (i.e. some iterator 
waiting on a resource but a close cancels the wait). 
   
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to