On May 22, 2007, at 10:42 PM, Ryan McKinley wrote:
If thats the case, I think the .diff you posted is fine...

Not really, because I commented out a bit to get past things. It was more than just setting the default to zero.

The only thing I would change is I think the default should be some positive number. For the app where you want the default to be 0, you can initialize the request handler with:

  <requestHandler ... >
    <lst name="defaults">
     <int name="numTerms">0</int>
    </lst>
  </requestHandler>

I don't get why the default should be non-zero. The most common use case would be field/type/size introspection, I presume. I don't see getting top terms being as needed. But, I'm fine with the default being non-zero if others feel it should be - setting it in the config file is no big deal for me :)

        Erik

Reply via email to