Skip to content

Use opaque id in task cancellation assertion #110680

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.

Sign up for GitHub

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

Merged
merged 8 commits into from
Jul 12, 2024
Merged

Use opaque id in task cancellation assertion #110680

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
0648030
narrow-down tasks assertion per http request
mhl-b Jul 9, 2024
37571a4
cleanup
mhl-b Jul 10, 2024
a4a4bd4
add assertions
mhl-b Jul 10, 2024
d7f5d3a
more tracing and error logs
mhl-b Jul 10, 2024
622c5b1
spotless
mhl-b Jul 10, 2024
a6a6fa2
feedback
mhl-b Jul 11, 2024
aa22373
Merge remote-tracking branch 'upstream/main' into assert-task-per-req…
mhl-b Jul 11, 2024
809296b
poke ci
mhl-b Jul 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
spotless
  • Loading branch information
@mhl-b
mhl-b committed Jul 10, 2024
commit 622c5b198c24d83c3d75dca7d9631c7c6e6f51e9
2 changes: 1 addition & 1 deletion test/framework/src/main/java/org/elasticsearch/test/TaskAssertions.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@
import java.util.concurrent.TimeUnit;
import java.util.stream.Collectors;

import static junit.framework.TestCase.assertTrue;
import static junit.framework.TestCase.assertFalse;
import static junit.framework.TestCase.assertTrue;
import static junit.framework.TestCase.fail;
import static org.elasticsearch.test.ESIntegTestCase.client;
import static org.elasticsearch.test.ESIntegTestCase.internalCluster;
Expand Down