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

   ### Description
   
   See #15212 for background.
   
   The "long tail" of slowest tests was marked `@Nightly` with a comment of `// 
incredibly slow` or similar with 5 commits:
   * 
https://github.com/apache/lucene/commit/86160952d9f2908f440c6d2327790f2309aea425
   * 
https://github.com/apache/lucene/commit/c3ef12f829285da4ac83cd97ba9f5003e7be1510
   * 
https://github.com/apache/lucene/commit/6c66a9d0ea9b4cde99accc832d6d998512c1f928
   * 
https://github.com/apache/lucene/commit/1aa9ed3f379c12de6cf5c26f35f02ee259e25a8a
   * 
https://github.com/apache/lucene/commit/7a663d68f1b2a71b309857fe49ddaf19f316ce5f
   
   This is not ideal: just a workaround to help keep the CI going. Instead we 
would prefer tests run quickly so that they can e.g. run when testing new PRs.
   
   We should fix the performance issues with these tests if possible. Maybe 
some truly need to remain `@Nightly`, but it should be the last resort. In many 
cases, the test might just be slow "some of the time" or consume too many 
resources (threads). For example, multithreaded cases using more than 2 
threads: it should not be needed. Other tests might be using too much 
data/memory/cpu cycles and just need to be toned down. Other ones might even be 
real performance issues, we have to look into them more to figure out.
   
   Feel free to open issue/PR for any of these tests if you have an idea how to 
fix them.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to