You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "selckin (Created) (JIRA)" <ji...@apache.org> on 2012/04/20 10:24:41 UTC
[jira] [Created] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
FieldCacheRewriteMethod.java is in tests
----------------------------------------
Key: LUCENE-4003
URL: https://issues.apache.org/jira/browse/LUCENE-4003
Project: Lucene - Java
Issue Type: Bug
Reporter: selckin
I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Commented] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
Posted by "Robert Muir (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/LUCENE-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258144#comment-13258144 ]
Robert Muir commented on LUCENE-4003:
-------------------------------------
this was added mainly to have a test for doctermsindex's termenum,
so i think src/test is the correct place.
> FieldCacheRewriteMethod.java is in tests
> ----------------------------------------
>
> Key: LUCENE-4003
> URL: https://issues.apache.org/jira/browse/LUCENE-4003
> Project: Lucene - Java
> Issue Type: Bug
> Reporter: selckin
>
> I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
> Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Commented] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
Posted by "selckin (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/LUCENE-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258145#comment-13258145 ]
selckin commented on LUCENE-4003:
---------------------------------
I found it because i was looking for a query equivalent to FieldCacheRangeFilter
> FieldCacheRewriteMethod.java is in tests
> ----------------------------------------
>
> Key: LUCENE-4003
> URL: https://issues.apache.org/jira/browse/LUCENE-4003
> Project: Lucene - Java
> Issue Type: Bug
> Reporter: selckin
>
> I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
> Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Commented] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
Posted by "Robert Muir (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/LUCENE-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258178#comment-13258178 ]
Robert Muir commented on LUCENE-4003:
-------------------------------------
Feel free to take this one Uwe!
{quote}
FieldCacheTermsFilter and also the TermsFilter in contrib can go away and we should instead use the sorted terms automaton (I always forget the name, Dawid, sorry) as a new AtomatonQuery subclass.
{quote}
That one should maybe be separated out... i could help some. Looking at the description of the thing, i think
in some situations it could be a speedup (imagine some of the terms dont exist at all, etc).
Also i wonder about the API, it could still have the current add() api, and do the sorting->automaton at the
end (e.g. rewrite), but imo thats wasteful since the automaton is independent of the reader, so instead maybe it
could take Term[] up front.
Finally, a new expert ctor (maybe just protected) should be exposed to AutomatonQuery:
currently you have:
{code}
public AutomatonQuery(final Term term, Automaton automaton) {
super(term.field());
this.term = term;
this.automaton = automaton;
this.compiled = new CompiledAutomaton(automaton);
}
{code}
But I would add AutomatonQuery(Term, Automaton, CompiledAutomaton), so that a subclass
could pass an already-compiled automaton. This TermsFilter-query would use the alternative
CompiledAutomaton ctor, passing true for finite, and false for simplify, since we know
its finite and probably large (so simplification in general would hurt).
> FieldCacheRewriteMethod.java is in tests
> ----------------------------------------
>
> Key: LUCENE-4003
> URL: https://issues.apache.org/jira/browse/LUCENE-4003
> Project: Lucene - Java
> Issue Type: Bug
> Reporter: selckin
>
> I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
> Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Commented] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
Posted by "Uwe Schindler (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/LUCENE-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258164#comment-13258164 ]
Uwe Schindler commented on LUCENE-4003:
---------------------------------------
I like this idea, too.
I would like to move this rewrite method (a little bit refactored, using FixedBitSet instead OpenBitSet) to core, factor out the inner Filter impl to be something like MTQWrapperFilter (maybe include the code in MTQWrapperFilter and only set a boolean in ctor). The rewrite mode would then be similar to FilterRewrite, just with another boolean.
We can remove FieldCacheRangeFilter.newStringRange() [or at least rewrite it to use this rewrite method], which would be elegant, too. FieldCacheTermsFilter and also the TermsFilter in contrib can go away and we should instead use the sorted terms automaton (I always forget the name, Dawid, sorry) as a new AtomatonQuery subclass.
> FieldCacheRewriteMethod.java is in tests
> ----------------------------------------
>
> Key: LUCENE-4003
> URL: https://issues.apache.org/jira/browse/LUCENE-4003
> Project: Lucene - Java
> Issue Type: Bug
> Reporter: selckin
>
> I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
> Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Commented] (LUCENE-4003) FieldCacheRewriteMethod.java is in
tests
Posted by "Robert Muir (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/LUCENE-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258149#comment-13258149 ]
Robert Muir commented on LUCENE-4003:
-------------------------------------
Thats a nice point... why do we have fieldcacherangefilter and fieldcachetermsfilter,
that only work with certain queries?!
This is a more general version, that works with any multitermquery...
> FieldCacheRewriteMethod.java is in tests
> ----------------------------------------
>
> Key: LUCENE-4003
> URL: https://issues.apache.org/jira/browse/LUCENE-4003
> Project: Lucene - Java
> Issue Type: Bug
> Reporter: selckin
>
> I believe FieldCacheRewriteMethod was accidentally moved to the wrong place in r1158697 and should be src/main and not in src/test.
> Or is it something that you don't want people to use?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org