[GitHub] [lucene] romseygeek commented on pull request #12156: Remove custom TermInSetQuery implementation in favor of extending MultiTermQuery

["romseygeek (via GitHub)" <gi...@apache.org>]

romseygeek commented on PR #12156:
URL: https://github.com/apache/lucene/pull/12156#issuecomment-1436004819

   > I'm not familiar enough with the monitor package to assess if it would be reasonable to just change the unit test, or if there's something more important going on here.
   
   It's fine to just change the test case here, we're only checking that the debug functionality on the monitor reports that a given query has matched and the specific string output isn't important.


-- 
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: issues-unsubscribe@lucene.apache.org

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