lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael McCandless <luc...@mikemccandless.com>
Subject Re: svn commit: r1152673 - /lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/Tes tPayloads.java
Date Mon, 01 Aug 2011 13:22:29 GMT
OK I found the problem: 3.x can't handle the U+FFFF character (we
replace it on indexing), while trunk can.  So I think we just have to
fix randomFixedByteLengthUnicodeString to never use that char.  I'll
commit...

Mike McCandless

http://blog.mikemccandless.com

On Sun, Jul 31, 2011 at 10:33 PM, Robert Muir <rcmuir@gmail.com> wrote:
> On Sun, Jul 31, 2011 at 10:24 PM, Chris Hostetter
> <hossman_lucene@fucit.org> wrote:
>>
>> : fix test to not create invalid unicode
>>
>> I'm confused ... when/why does randomFixedByteLengthUnicodeString not
>> return valid unicode?
>>
>
> I'm confused too, but all of the issues are with replacement chars for
> invalid unicode:
>
> Since the intent of this test is to test thread safety, not to test
> unicode enc/dec back and forth, I switched it to ascii until the test
> grows up (e.g. trunk, which now uses full unicode range correctly,
> maybe I backported this wrong before)
>
> --
> lucidimagination.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: dev-help@lucene.apache.org
>
>

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


Mime
View raw message