[jira] Updated: (LUCENE-1929) Highlighter doesn't support NumericQuery or deprecated RangeQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Miller updated LUCENE-1929: Description: Sucks. Will throw a NullPointer exception. Only NumericRangeQuery will throw the ex

[jira] Updated: (LUCENE-1929) Highlighter doesn't support NumericQuery or deprecated RangeQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Miller updated LUCENE-1929: Summary: Highlighter doesn't support NumericQuery or deprecated RangeQuery (was: Highlighter does

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760076#action_12760076 ] Mark Miller commented on LUCENE-1929: - bq. Another option to fix Solr's highlighter is

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760074#action_12760074 ] Uwe Schindler commented on LUCENE-1929: --- Another option to fix Solr's highlighter is

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760072#action_12760072 ] Mark Miller commented on LUCENE-1929: - bq. I wasn't blaming you for the problem In fa

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760071#action_12760071 ] Mark Miller commented on LUCENE-1929: - Ah, thanks - didn't catch that. Wondered how it

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760070#action_12760070 ] Uwe Schindler commented on LUCENE-1929: --- bq. On a side note, its a bummer we seem to

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760068#action_12760068 ] Mark Miller commented on LUCENE-1929: - bq. and not a backwards break, as both classes

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760067#action_12760067 ] Mark Miller commented on LUCENE-1929: - I realize the difficulties - I wasn't blaming y

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760066#action_12760066 ] Uwe Schindler commented on LUCENE-1929: --- bq. In fact, its probably wrong - but at th

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760064#action_12760064 ] Mark Miller commented on LUCENE-1929: - bq. so whether return null from getTerm is righ

Re: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Mark Miller
Yeah, I agree Hoss - we shouldn't pull the rug in either case - if you could count on it before and its worked out well, its more of a hassle to push out a surprise than to keep the known behavior. More of an aside from me ;) Chris Hostetter wrote: > : That is my opinion, too. Closing the readers

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760062#action_12760062 ] Mark Miller commented on LUCENE-1929: - Yeah - eventually (as in when getTerm() is remo

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Chris Hostetter
: That is my opinion, too. Closing the readers should be done by the caller in I don't disagree with either of you, but... : a finally block and not automatically by the IW. I only wanted to confirm, : that the behaviour of 2.9 did not change. Closing readers two times is not a ...i wanted to t

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760061#action_12760061 ] Uwe Schindler commented on LUCENE-1929: --- I read what stands in SOLR-1221: It calls M

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760060#action_12760060 ] Uwe Schindler commented on LUCENE-1929: --- Does it throw a NPE for every "unknown" que

[jira] Commented: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760056#action_12760056 ] Mark Miller commented on LUCENE-1929: - So making NumericQuery Hilightable is not going

[jira] Created: (LUCENE-1929) Highlighter doesn't support NumericQuery

2009-09-27 Thread Mark Miller (JIRA)
Highlighter doesn't support NumericQuery Key: LUCENE-1929 URL: https://issues.apache.org/jira/browse/LUCENE-1929 Project: Lucene - Java Issue Type: Bug Components: contrib/highlighter Af

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Uwe Schindler
> I guess the fact that Lucene closes it is a legacy we may be stuck with > - but I think it encourages bad practice code. There is plenty of room > for an exception to be thrown before the Reader is sure to be closed in > a finally block. > > Part of why its standard convention to close your own

Re: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Mark Miller
I guess the fact that Lucene closes it is a legacy we may be stuck with - but I think it encourages bad practice code. There is plenty of room for an exception to be thrown before the Reader is sure to be closed in a finally block. Part of why its standard convention to close your own I think, rat

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Chris Hostetter
: So in 2.9, the Reader is correctly closed, if the TokenStream chain is : correctly set up, passing all close() calls to the delegate. Thanks for digging into that Uwe. So Daniel: The ball is in your court here: what analyzer / tokenizer+tokenfilters is your app using in the cases where you se

[jira] Commented: (LUCENE-1458) Further steps towards flexible indexing

2009-09-27 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760031#action_12760031 ] Michael McCandless commented on LUCENE-1458: The codec is per segment. Howeve

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Uwe Schindler
Sorry my last reply was nonsense. Even if you reuse TokenStreams, the consumer should close the stream at the end of operations. And this is for sure done by the DocInverterPerField (finally-block within TokenStream.close()). Maybe the user with the problem has created a TokenFilter or something l

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Uwe Schindler
I think I know, what is different than earlier versions: The close of the Reader is done by Tokenizer.close(). As in 2.9.0 we did much work to get all the Tokenizers reusable, there is the following problem: The Reader is only closed, if you call Tokenizer.close(), but not, if you call Tokenizer.

RE: Lucene 2.9.0-rc5 : Reader stays open after IndexWriter.updateDocument(), is that possible?

2009-09-27 Thread Uwe Schindler
How does your test perform with 2.4.1 ? - Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de eMail: [email protected] > -Original Message- > From: Chris Hostetter [mailto:[email protected]] > Sent: Sunday, September 27, 2009 3:34 AM > To: Lucene Dev > Subj