[LUCENE-6328] Make Filter.clone and Filter.setBoost throw an UnsupportedOperationException

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Won't Fix
• Lucene Fields: New
• Assignee: Adrien Grand
• Reporter: Adrien Grand
• Created: 03/Mar/15 13:07
• Updated: 28/Aug/22 14:28
• Resolved: 29/Apr/15 07:37

Description

The rewrite process uses a combination of calls to clone() and setBoost(boost) in order to rewrite queries. This is a bit weird for filters given that they were not originally designed to care about scoring.

Using a filter directly as a query fails unit tests today since filters do not pass the QueryUtils checks: it is expected that cloning and changing the boost results in an instance which is unequal. However existing filters do not take into account the getBoost() parameter inherited from Query so this test fails.

I think it would be less error-prone to throw an UnsupportedOperationException for clone() and setBoost() on filters and disable the check in QueryUtils for filters.

In order to keep rewriting working, filters could rewrite to a CSQ around themselves so that clone() and setBoost() would be called on the CSQ.

Issue Links

• is superceded by: LUCENE-6448 Make Filter a better Query citizen