[ 
https://issues.apache.org/jira/browse/LUCENE-5991?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160085#comment-14160085
 ] 

Michael McCandless commented on LUCENE-5991:
--------------------------------------------

+1

> add minor-revision writers to backwards-codecs
> ----------------------------------------------
>
>                 Key: LUCENE-5991
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5991
>             Project: Lucene - Core
>          Issue Type: Test
>            Reporter: Robert Muir
>
> Today we have backwards testing almost completely isolated cleanly, and tests 
> against each format. But we only test old major formats, not the minor ones. 
> Before it was probably the right tradeoff, but now that its isolated I think 
> we should test all of them.
> For example the 4.1 stored fields format had two minor format changes across 
> the 4.x release: 
> {noformat}
>   static final int VERSION_START = 0;
>   static final int VERSION_BIG_CHUNKS = 1;
>   static final int VERSION_CHECKSUM = 2;
>   static final int VERSION_CURRENT = VERSION_CHECKSUM;
> {noformat}
> We could easily directly test these possibilities (e.g. take this as a 
> parameter to the RW format and have 3 TestXXXStoredFieldsFormat, one for 
> each) instead of only testing the latest one and relying on TestBackCompat to 
> find issues, which it probably won't since the index is simplistic.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to