lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yonik Seeley <yo...@lucidimagination.com>
Subject Re: NPE on startup when you comment out the queries in firstSearcher
Date Sun, 12 Jun 2011 13:37:54 GMT
On Sun, Jun 12, 2011 at 9:03 AM, Erick Erickson <erickerickson@gmail.com> wrote:
> I think this has been around forever (I'm seeing it in 1.4.1 too), but
> I just happened to notice it. If you comment out the <arr
> name="queries"> section in firstSearcher, you get an NPE as below:
>
> Any reason I shouldn't make a patch and even apply it when I get my
> credentials? It's a trivial fix, just return if the list is null.

+1
Since the effect is to get rid of an exception in the logs, this
doesn't really even need a CHANGES entry or anything IMO.
Commit away when you can!  Then merge it back to the 3x branch to
brush up on your svn skills ;-)

-Yonik
http://www.lucidimagination.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message