[ 
https://issues.apache.org/jira/browse/LUCENENET-226?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Garski updated LUCENENET-226:
-------------------------------------

    Attachment: TestFieldSanityChecker.patch

Got it!  All tests now pass.

Here's a summary of the changes:

Lucene.Net/Util/AverageGuessMemoryModel.cs : had an entry in the size lookup 
for sbyte, not byte

Lucene.Net/Util/MapOfSets.cs : reworked to use generics as it going through all 
of the iterators in FieldSanityChecker was giving me the spins.  As HashSet<T> 
is only available in the 3.5 version of the framework (looking forward to 
deprecting 2.0) I used a Dictionary<T,T> to mimic it.

Lucene.Net/Util/FieldCacheSanityChecker.cs : uses the generic MapOfSets.  There 
were issues with the enumerators and using foreach's makes it easier to use and 
debug.

Test/Util/TestFieldCacheSanityChecker.cs : minor change in how the doubles are 
indexed, ensuring the string format supported round-trip for Double.MaxValue

Feedback this way, please!

> Tests in Lucene.Net.Util.TestFieldCacheSanityChecker
> ----------------------------------------------------
>
>                 Key: LUCENENET-226
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-226
>             Project: Lucene.Net
>          Issue Type: Bug
>            Reporter: Michael Garski
>         Attachments: TestFieldSanityChecker.patch
>
>
> There are 3 tests failing at this time:
> Lucene.Net.Util.TestFieldCacheSanityChecker.TestInsanity1
> Lucene.Net.Util.TestFieldCacheSanityChecker.TestInsanity2
> Lucene.Net.Util.TestFieldCacheSanityChecker.TestSanity
> I'll be working with those and updating this issue with progress or issues.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to