[jira] [Updated] (LUCENE-7078) Make remaining mutable Queries immutable
[ https://issues.apache.org/jira/browse/LUCENE-7078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ishan Chattopadhyaya updated LUCENE-7078: - Fix Version/s: (was: 6.4) 6.5 > Make remaining mutable Queries immutable > > > Key: LUCENE-7078 > URL: https://issues.apache.org/jira/browse/LUCENE-7078 > Project: Lucene - Core > Issue Type: Improvement > Components: core/search >Reporter: Luc Vanlerberghe >Priority: Minor > Fix For: master (7.0), 6.5 > > > See LUCENE-6531 > Mutable queries are an issue for automatic filter caching since modifying a > query after it has been put into the cache will corrupt the cache. We should > make all queries immutable (up to the boost) to avoid this issue. > Since they are part of the public API I would suggest splitting them in an > immutable class and a builder like was done for most other Queries *before* > releasing an official 6.x version > I did a quick scan through all derived classes of Query and I compiled the > following list (ignoring sources in test or contrib folders) > Some of them are already marked as experimental (but should perhaps receive > the "official" @lucene.experimental tag ?) > For some it's possibly not an issue since they should never end up in a > filter cache (like MoreLikeThisQuery ?), but then a comment specifying the > exception to the rule should perhaps be added. > * lucene/queries: > ** org.apache.lucene.queries.CommonTermsQuery > ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) > ** org.apache.lucene.queries.mlt.MoreLikeThisQuery > * lucene/suggest: > ** org.apache.lucene.search.suggest.document.ContextQuery (marked as > @lucene.experimental) > * lucene/facet: > ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Updated] (LUCENE-7078) Make remaining mutable Queries immutable
[ https://issues.apache.org/jira/browse/LUCENE-7078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shalin Shekhar Mangar updated LUCENE-7078: -- Fix Version/s: (was: 6.3) 6.4 > Make remaining mutable Queries immutable > > > Key: LUCENE-7078 > URL: https://issues.apache.org/jira/browse/LUCENE-7078 > Project: Lucene - Core > Issue Type: Improvement > Components: core/search >Reporter: Luc Vanlerberghe >Priority: Minor > Fix For: master (7.0), 6.4 > > > See LUCENE-6531 > Mutable queries are an issue for automatic filter caching since modifying a > query after it has been put into the cache will corrupt the cache. We should > make all queries immutable (up to the boost) to avoid this issue. > Since they are part of the public API I would suggest splitting them in an > immutable class and a builder like was done for most other Queries *before* > releasing an official 6.x version > I did a quick scan through all derived classes of Query and I compiled the > following list (ignoring sources in test or contrib folders) > Some of them are already marked as experimental (but should perhaps receive > the "official" @lucene.experimental tag ?) > For some it's possibly not an issue since they should never end up in a > filter cache (like MoreLikeThisQuery ?), but then a comment specifying the > exception to the rule should perhaps be added. > * lucene/queries: > ** org.apache.lucene.queries.CommonTermsQuery > ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) > ** org.apache.lucene.queries.mlt.MoreLikeThisQuery > * lucene/suggest: > ** org.apache.lucene.search.suggest.document.ContextQuery (marked as > @lucene.experimental) > * lucene/facet: > ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Updated] (LUCENE-7078) Make remaining mutable Queries immutable
[ https://issues.apache.org/jira/browse/LUCENE-7078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicholas Knize updated LUCENE-7078: --- Fix Version/s: 6.1 trunk > Make remaining mutable Queries immutable > > > Key: LUCENE-7078 > URL: https://issues.apache.org/jira/browse/LUCENE-7078 > Project: Lucene - Core > Issue Type: Improvement > Components: core/search >Reporter: Luc Vanlerberghe >Priority: Minor > Fix For: trunk, 6.1 > > > See LUCENE-6531 > Mutable queries are an issue for automatic filter caching since modifying a > query after it has been put into the cache will corrupt the cache. We should > make all queries immutable (up to the boost) to avoid this issue. > Since they are part of the public API I would suggest splitting them in an > immutable class and a builder like was done for most other Queries *before* > releasing an official 6.x version > I did a quick scan through all derived classes of Query and I compiled the > following list (ignoring sources in test or contrib folders) > Some of them are already marked as experimental (but should perhaps receive > the "official" @lucene.experimental tag ?) > For some it's possibly not an issue since they should never end up in a > filter cache (like MoreLikeThisQuery ?), but then a comment specifying the > exception to the rule should perhaps be added. > * lucene/queries: > ** org.apache.lucene.queries.CommonTermsQuery > ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) > ** org.apache.lucene.queries.mlt.MoreLikeThisQuery > * lucene/suggest: > ** org.apache.lucene.search.suggest.document.ContextQuery (marked as > @lucene.experimental) > * lucene/facet: > ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Updated] (LUCENE-7078) Make remaining mutable Queries immutable
[ https://issues.apache.org/jira/browse/LUCENE-7078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Luc Vanlerberghe updated LUCENE-7078: - Description: See LUCENE-6531 Mutable queries are an issue for automatic filter caching since modifying a query after it has been put into the cache will corrupt the cache. We should make all queries immutable (up to the boost) to avoid this issue. Since they are part of the public API I would suggest splitting them in an immutable class and a builder like was done for most other Queries *before* releasing an official 6.x version I did a quick scan through all derived classes of Query and I compiled the following list (ignoring sources in test or contrib folders) Some of them are already marked as experimental (but should perhaps receive the "official" @lucene.experimental tag ?) For some it's possibly not an issue since they should never end up in a filter cache (like MoreLikeThisQuery ?), but then a comment specifying the exception to the rule should perhaps be added. * lucene/queries: ** org.apache.lucene.queries.CommonTermsQuery ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) ** org.apache.lucene.queries.mlt.MoreLikeThisQuery * lucene/suggest: ** org.apache.lucene.search.suggest.document.ContextQuery (marked as @lucene.experimental) * lucene/facet: ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) was: See LUCENE-6531 Mutable queries are an issue for automatic filter caching since modifying a query after it has been put into the cache will corrupt the cache. We should make all queries immutable (up to the boost) to avoid this issue. Since they are part of the public API I would suggest splitting them in an immutable class and a builder like was done for most other Queries *before* releasing an official 6.x version I did a quick scan through all derived classes of Query and I compiled the following list (ignoring sources in test or contrib folders) Some of them are already marked as experimental (but should perhaps receive the "official" @lucene.experimental tag ?) For some it's possibly not an issue since they should never end up in a filter cache (like MoreLikeThisQuery ?), but then a comment specifying the exception to the rule should perhaps be added. * lucene/search: ** org.apache.lucene.search.MultiPhraseQuery * lucene/queries: ** org.apache.lucene.queries.CommonTermsQuery ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) ** org.apache.lucene.queries.mlt.MoreLikeThisQuery * lucene/suggest: ** org.apache.lucene.search.suggest.document.ContextQuery (marked as @lucene.experimental) * lucene/facet: ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) > Make remaining mutable Queries immutable > > > Key: LUCENE-7078 > URL: https://issues.apache.org/jira/browse/LUCENE-7078 > Project: Lucene - Core > Issue Type: Improvement > Components: core/search >Reporter: Luc Vanlerberghe >Priority: Minor > > See LUCENE-6531 > Mutable queries are an issue for automatic filter caching since modifying a > query after it has been put into the cache will corrupt the cache. We should > make all queries immutable (up to the boost) to avoid this issue. > Since they are part of the public API I would suggest splitting them in an > immutable class and a builder like was done for most other Queries *before* > releasing an official 6.x version > I did a quick scan through all derived classes of Query and I compiled the > following list (ignoring sources in test or contrib folders) > Some of them are already marked as experimental (but should perhaps receive > the "official" @lucene.experimental tag ?) > For some it's possibly not an issue since they should never end up in a > filter cache (like MoreLikeThisQuery ?), but then a comment specifying the > exception to the rule should perhaps be added. > * lucene/queries: > ** org.apache.lucene.queries.CommonTermsQuery > ** org.apache.lucene.queries.CustomScoreQuery (marked as @lucene.experimental) > ** org.apache.lucene.queries.mlt.MoreLikeThisQuery > * lucene/suggest: > ** org.apache.lucene.search.suggest.document.ContextQuery (marked as > @lucene.experimental) > * lucene/facet: > ** org.apache.lucene.facet.DrillDownQuery (marked as @lucene.experimental) -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
