Re: Should Document.getFieldables really return null

2008-03-14 Thread Stefan Trcek
On Friday 14 March 2008 11:46:42 Michael McCandless wrote: > I agree, this makes sense. I'll commit it. Thanks Stefan! > > Except, the last one you list (getBinaryValue) I think should still > return null if no field by that name exists? Yes, you are right. Looking at the array notion made me so

Re: Should Document.getFieldables really return null

2008-03-14 Thread Michael McCandless
I agree, this makes sense. I'll commit it. Thanks Stefan! Except, the last one you list (getBinaryValue) I think should still return null if no field by that name exists? Mike Stefan Trcek wrote: Hello The 'Document.getFieldables(String name)' is documented to return 'null' in some c

Should Document.getFieldables really return null

2008-03-13 Thread Stefan Trcek
Hello The 'Document.getFieldables(String name)' is documented to return 'null' in some cases (and really does, see the code below). However this makes a penalty to the client, as code like this Document doc = hits.doc(i); for (Fieldable f: doc.getFieldables("somefield")) { Syst