jainankitk commented on PR #13711: URL: https://github.com/apache/lucene/pull/13711#issuecomment-2327098073
> I see from the linked issue that you would like to extend `PointRangeQuery`, but in general we don't like to think of our queries as being extensible. I wonder if you could do what you need through composition rather than extension? @jpountz - Thanks for your feedback. While I agree with using composition rather than extension for `PointRangeQuery` itself, `Weight` implementation itself within `PointRangeQuery` will be more reusable (via composition instead of extension) by being a non-anonymous class. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
