ChrisHegarty commented on PR #15319: URL: https://github.com/apache/lucene/pull/15319#issuecomment-3385888650
> but maybe it's too much voodoo. Yeah, I consider it too much voodoo to add in our source base, given that the source code change is so trivial. I add a comment in the code which notes to not refactor this class in such a way that can break things. This should be enough until we get the static analysis. My primary motivation for mentioning the voodoo workaround is to help users of Lucene if they run into this problem - until we can get the fix into 10.4. For example, we've seen this deadlock several times in Elasticsearch, but only coming to my attention now. So we'll apply the voodoo workaround there until the next 10.x release. -- 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]
