XComp commented on code in PR #20805:
URL: https://github.com/apache/flink/pull/20805#discussion_r1000260540
##########
flink-test-utils-parent/flink-test-utils-junit/src/main/java/org/apache/flink/testutils/junit/extensions/retry/strategy/RetryStrategy.java:
##########
@@ -27,11 +27,17 @@ public interface RetryStrategy {
void stopFollowingAttempts();
/**
- * Handle the exception after the test execution.
+ * Handle an exception that occurred during the annotated test attempt.
+ *
+ * <p>This method can swallow the exception to pass the test.
*
* @param testName the test name
* @param attemptIndex test attempt index that starts from 1
* @param throwable the throwable that the test case throws
+ * @throws org.opentest4j.TestAbortedException When handling a test
attempt failure, throwing
Review Comment:
Ah true, I haven't considered that it's outside of the Flink space. :+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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
