You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2022/06/27 23:51:57 UTC
[GitHub] [lucene] madrob opened a new pull request, #989: Add back-compat indices for 8.11.2
madrob opened a new pull request, #989:
URL: https://github.com/apache/lucene/pull/989
Regenerated the index manually, not using the wizard. Spent a lot of time trying to isolate the failures, but couldn't figure them out. New index seems to work but I would appreciate other folks testing it.
Generated from a download of `lucene-8.11.2-src.tgz` with ant 1.9 and java 8.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene] madrob merged pull request #989: Add back-compat indices for 8.11.2
Posted by GitBox <gi...@apache.org>.
madrob merged PR #989:
URL: https://github.com/apache/lucene/pull/989
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene] jpountz commented on pull request #989: Add back-compat indices for 8.11.2
Posted by GitBox <gi...@apache.org>.
jpountz commented on PR #989:
URL: https://github.com/apache/lucene/pull/989#issuecomment-1168391661
What sort of failures did you get? I wonder if it would make things easier to copy indexes that have been produced on `branch_8_11` instead of regenerating them from the `main` branch.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene] madrob commented on pull request #989: Add back-compat indices for 8.11.2
Posted by GitBox <gi...@apache.org>.
madrob commented on PR #989:
URL: https://github.com/apache/lucene/pull/989#issuecomment-1168939508
Both the old index in d64900e and the regenerated index in f158be9 came off of branch_8_11, and more specifically the 8.11.2 release tag. I don't understand why they are different though. Old index was generated using commands in the release wizard and copied over from the commit in 963814fc454, regenerated index was built by running directly running `ant`, then zipped.
The error that we get with the "bad" index can be reproduced on `d64900e` with `./gradlew test --tests TestBackwardsCompatibility.testSortedIndex`
```
org.apache.lucene.backward_index.TestBackwardsCompatibility > testSortedIndex FAILED
java.lang.AssertionError
at __randomizedtesting.SeedInfo.seed([80EE95A3F6D9343B:F771F355CE204DEF]:0)
at junit@4.13.1/org.junit.Assert.fail(Assert.java:87)
at junit@4.13.1/org.junit.Assert.assertTrue(Assert.java:42)
at junit@4.13.1/org.junit.Assert.assertTrue(Assert.java:53)
at org.apache.lucene.backward_index.TestBackwardsCompatibility.searchExampleIndex(TestBackwardsCompatibility.java:2179)
at org.apache.lucene.backward_index.TestBackwardsCompatibility.testSortedIndex(TestBackwardsCompatibility.java:2148)
```
The test code in question is these lines:
```
topDocs = searcher.search(new TermQuery(new Term("body", "the")), 5);
assertTrue(topDocs.totalHits.value > 0);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org