[GitHub] [lucene-solr] ErickErickson commented on pull request #1668: SOLR-13939: Extract any non-gradle related patches (deprecations, URL…

[GitBox <gi...@apache.org>]

ErickErickson commented on pull request #1668:
URL: https://github.com/apache/lucene-solr/pull/1668#issuecomment-657811914


   @risdenk  Thanks for looking. This is more about adding thread leak detection to a bunch of classes. When Mark was working on Gradle, a lot of other stuff was changed that weren't strictly related to changing the build system and this JIRA is just about taking any of those changes and making them their own.
   
   Looking over some recent changes, I doubt this'll really cut down on the number of failures due to thread leak detection. OTOH, I don't see any harm in it.
   
   On the other other hand, I have to check inheritance. SolrIgnoredThreadsFilter is included in SolrTestCase, so IDK whether it's necessary to include it in anything that inherits from it... Siiiggghhh.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org