[ 
https://issues.apache.org/jira/browse/LUCENE-819?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12476664
 ] 

Dan Zivkovic commented on LUCENE-819:
-------------------------------------

Empty string or null would be the same thing from the point of view of our 
application. And it would be preferable to an exception coming from a place 
that didn't use to throw one.

I still don't see the big deal in checking for null, however. If the fix is 
simple and it's tested and it doesn't affect anyone who does NOT have nulls in 
their terms, then why not just apply the patch and move on?

> NullPointerException in Term.compareTo()
> ----------------------------------------
>
>                 Key: LUCENE-819
>                 URL: https://issues.apache.org/jira/browse/LUCENE-819
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Index
>         Environment: JDK 1.4 on MacOS
>            Reporter: Dan Zivkovic
>         Attachments: lucene-819.patch, lucene-819.patch, TestTerm.java
>
>
>  java.lang.NullPointerException
>       at org.apache.lucene.index.Term.compareTo(Term.java:84)
>       at 
> org.apache.lucene.index.TermInfosReader.getIndexOffset(TermInfosReader.java:115)
>       at org.apache.lucene.index.TermInfosReader.get(TermInfosReader.java:150)
>       at org.apache.lucene.index.SegmentReader.docFreq(SegmentReader.java:300)
>       at org.apache.lucene.search.IndexSearcher.docFreq(IndexSearcher.java:78)
>       at org.apache.lucene.search.Similarity.idf(Similarity.java:255)
>       at 
> org.apache.lucene.search.TermQuery$TermWeight.<init>(TermQuery.java:43)
>       at org.apache.lucene.search.TermQuery.createWeight(TermQuery.java:142)
>       at 
> org.apache.lucene.search.BooleanQuery$BooleanWeight.<init>(BooleanQuery.java:203)
>       at 
> org.apache.lucene.search.BooleanQuery$BooleanWeight2.<init>(BooleanQuery.java:330)
>       at 
> org.apache.lucene.search.BooleanQuery.createWeight(BooleanQuery.java:372)
>       at org.apache.lucene.search.Query.weight(Query.java:93)
>       at org.apache.lucene.search.Hits.<init>(Hits.java:48)
>       at org.apache.lucene.search.Searcher.search(Searcher.java:53)
>         ...

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to