[jira] [Updated] (LUCENE-6213) Add test for IndexFormatTooOldException if a commit has a 3.x segment

2015-02-02 Thread Ryan Ernst (JIRA)

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

Ryan Ernst updated LUCENE-6213:
---
Attachment: LUCENE-6213.patch

Sure, new patch with initCause added.

 Add test for IndexFormatTooOldException if a commit has a 3.x segment
 -

 Key: LUCENE-6213
 URL: https://issues.apache.org/jira/browse/LUCENE-6213
 Project: Lucene - Core
  Issue Type: Bug
Reporter: Robert Muir
 Attachments: LUCENE-6213.patch, LUCENE-6213.patch, LUCENE-6213.patch, 
 unsupported.4x-with-3x-segments.zip


 We should add a 4.x index (4.x commit) with some 3.x segment(s) to our 
 backwards tests.
 I don't think we throw IndexFormatTooOldException correctly in this case. I 
 think instead the user will get a confusing SPI error about a missing codec 
 Lucene3x.



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

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



[jira] [Updated] (LUCENE-6213) Add test for IndexFormatTooOldException if a commit has a 3.x segment

2015-02-02 Thread Ryan Ernst (JIRA)

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

Ryan Ernst updated LUCENE-6213:
---
Attachment: LUCENE-6213.patch

I tried the codec idea, but it unfortunately required changing a lot of callers 
to handling IOException or themselves throw IOException, and the change got 
very large.

Here is a new patch (still against branch_5x) that incorporates an additional 
check to give a nice error message if the user should probably include 
backward-codecs.jar.  It also is setup to easily forward port to trunk (just 
adding to the list of unsupportedCodecs).

 Add test for IndexFormatTooOldException if a commit has a 3.x segment
 -

 Key: LUCENE-6213
 URL: https://issues.apache.org/jira/browse/LUCENE-6213
 Project: Lucene - Core
  Issue Type: Bug
Reporter: Robert Muir
 Attachments: LUCENE-6213.patch, LUCENE-6213.patch, 
 unsupported.4x-with-3x-segments.zip


 We should add a 4.x index (4.x commit) with some 3.x segment(s) to our 
 backwards tests.
 I don't think we throw IndexFormatTooOldException correctly in this case. I 
 think instead the user will get a confusing SPI error about a missing codec 
 Lucene3x.



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

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



[jira] [Updated] (LUCENE-6213) Add test for IndexFormatTooOldException if a commit has a 3.x segment

2015-01-30 Thread Ryan Ernst (JIRA)

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

Ryan Ernst updated LUCENE-6213:
---
Attachment: unsupported.4x-with-3x-segments.zip
LUCENE-6213.patch

Here is a quick and dirty patch (against branch_5x) that adds a bwc index test 
as you suggested, and a quick fix for the bug.  I like the idea of a dummy 
codec, but didn't have time to try it.

 Add test for IndexFormatTooOldException if a commit has a 3.x segment
 -

 Key: LUCENE-6213
 URL: https://issues.apache.org/jira/browse/LUCENE-6213
 Project: Lucene - Core
  Issue Type: Bug
Reporter: Robert Muir
 Attachments: LUCENE-6213.patch, unsupported.4x-with-3x-segments.zip


 We should add a 4.x index (4.x commit) with some 3.x segment(s) to our 
 backwards tests.
 I don't think we throw IndexFormatTooOldException correctly in this case. I 
 think instead the user will get a confusing SPI error about a missing codec 
 Lucene3x.



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

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