You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "zhaih (via GitHub)" <gi...@apache.org> on 2023/09/17 00:06:32 UTC

[GitHub] [lucene] zhaih opened a new issue, #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

zhaih opened a new issue, #12562:
URL: https://github.com/apache/lucene/issues/12562

   ### Description
   
   Build: https://ci-builds.apache.org/job/Lucene/job/Lucene-Check-main/10242/
   
   1 tests failed.
   FAILED:  org.apache.lucene.search.matchhighlight.TestPassageSelector.randomizedSanityCheck
   
   Error Message:
   ```
   java.lang.AssertionError:
   Expected: not a collection containing "><"
        but: was <[?, B'nE?
   , ><, |u, $HX?Ox@, fV}?, ]ZR??x9]>
   
   Stack Trace:
   java.lang.AssertionError:
   Expected: not a collection containing "><"
        but: was <[?, B'nE?
   , ><, |u, $HX?Ox@, fV}?, ]ZR??x9]>
           at __randomizedtesting.SeedInfo.seed([BA589FCC77C2F999:4E6DB4BADBADAB93]:0)
           at org.hamcrest@2.2/org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
           at org.hamcrest@2.2/org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
           at org.apache.lucene.search.matchhighlight.TestPassageSelector.randomizedSanityCheck(TestPassageSelector.java:350)
   ...
   ```
   Reproduce with: 
   ```
   gradlew :lucene:highlighter:test --tests "org.apache.lucene.search.matchhighlight.TestPassageSelector.randomizedSanityCheck" -Ptests.jvms=4 -Ptests.haltonfailure=false -Ptests.jvmargs= -Ptests.seed=BA589FCC77C2F999 -Ptests.multiplier=2 -Ptests.gui=true -Ptests.file.encoding=ISO-8859-1
   ```
   
   I can reproduce it on both main and branch_9x
   
   ### Gradle command to reproduce
   
   :lucene:highlighter:test --tests "org.apache.lucene.search.matchhighlight.TestPassageSelector.randomizedSanityCheck" -Ptests.jvms=4 -Ptests.haltonfailure=false -Ptests.jvmargs= -Ptests.seed=BA589FCC77C2F999 -Ptests.multiplier=2 -Ptests.gui=true -Ptests.file.encoding=ISO-8859-1


-- 
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.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] dweiss commented on issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #12562:
URL: https://github.com/apache/lucene/issues/12562#issuecomment-1722957920

   The "build success" here is caused by an explicit change made by Uwe, here: https://github.com/apache/lucene/issues/10513#issuecomment-1224072458
   
   In his own words:
   
   The tests.haltonfailure was added back, as it was heavily used by Jenkins. [...] Jenkins wants to differentiate between "build passed, but test failures" vs. "build did not pass due to compile failure, verifications". haltonfailure just reconfigures (like in ant) that the test runner does not exit the whole build with non-success exit code. Jenkins does not rely on exit code, it monitors test result xml files.
   
   This makes sense in a CI world. Maybe we should omit printing this flag in the "reproduce with" line though. I'll file a separate issue for that.


-- 
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] dweiss commented on issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #12562:
URL: https://github.com/apache/lucene/issues/12562#issuecomment-1722400116

   Thanks @zhaih - this looks like something I wrote... I'll be taking a look on Monday, please feel free to leave it to me.


-- 
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] dweiss commented on issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #12562:
URL: https://github.com/apache/lucene/issues/12562#issuecomment-1722400642

   Also, I'm not sure why the build indeed resulted in "build success" - will look into that as well,.
   ```
   /home/jenkins/jenkins-slave/workspace/Lucene/Lucene-Check-main/gradlew -Dtests.haltonfailure=false -Dtests.badapples=false --continue check
   ```


-- 
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] dweiss closed issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss closed issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error
URL: https://github.com/apache/lucene/issues/12562


-- 
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] dweiss commented on issue #12562: search.matchhighlight.TestPassageSelector.randomizedSanityCheck reproducible error

Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #12562:
URL: https://github.com/apache/lucene/issues/12562#issuecomment-1723003992

   The issue is caused by random input clashing with test assertion, not a problem. I've filed PRs for both problems.


-- 
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