Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SOLR-16304 De-annotate some slow tests #951

Merged
merged 33 commits into from Aug 2, 2022
Merged

SOLR-16304 De-annotate some slow tests #951

merged 33 commits into from Aug 2, 2022

Conversation

madrob
Copy link
Contributor

@madrob madrob commented Jul 22, 2022

Removed Slow annotation from anything that ran faster than 15s on my machine.
Also removed Slow annotation from anything that was already Nightly

https://issues.apache.org/jira/browse/SOLR-16304

Removed Slow annotation from anything that ran faster than 15s on my machine.
Also removed Slow annotation from anything that was already Nightly
@madrob madrob requested review from epugh and dsmiley July 22, 2022 17:30
@madrob madrob requested a review from cpoerschke July 25, 2022 21:24
Copy link
Contributor

@dsmiley dsmiley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

@dsmiley
Copy link
Contributor

dsmiley commented Aug 2, 2022

Just checking; is "de-annotate some slow tests" still accurate -- thus it is still used but just not on tests that are not sufficiently slow?

@madrob
Copy link
Contributor Author

madrob commented Aug 2, 2022

Slow is no longer used at all

@madrob madrob merged commit d41ae28 into apache:main Aug 2, 2022
madrob added a commit that referenced this pull request Aug 2, 2022
Removed @LuceneTestCase.Slow test annotation from our code base
Some tests were re-annotated as Nightly tests because they are too long

Speed up tests:
* Scope down work in non-nightly runs and full testing in Nightly mode
* Combined use of cluster in single classes to speed up multiple methods
* Replaced several polling sleeps with zookeeper watches
* Replaced slow waits on async calls with sync calls when not explicitly testing async

(cherry picked from commit d41ae28)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants