Re: Fix to contrib/misc/HighFreqTerms.java

2010-04-17 Thread Michael McCandless
Ahh you're right! Though, really, we should not be converting to String (flex terms in general are an arbitrary byte[], not necessarily utf8). We should just use a BytesRef directly in the key. Can you open an issue for this Tom? Thanks! Mike On Fri, Apr 16, 2010 at 2:41 PM, Burton-West, Tom

RE: Fix to contrib/misc/HighFreqTerms.java

2010-04-16 Thread Burton-West, Tom
Hi Mike, Thanks for making the fix and changing the display from bytes to utf8. It needs a very minor change: The latest fix converts to utf8 if you give a field argument on the command line but still shows bytes if you don't. Line 89 should parallel line 70 and use term.utf8ToString() instead