RyanSkraba commented on code in PR #20805:
URL: https://github.com/apache/flink/pull/20805#discussion_r1000259397
##########
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:
:D Luckily the `@throws` annotation automatically links to the exception
when it makes sense. In this case, TestAbortedException is outside the
generated Flink javadocs in any case, I'm not sure external links to Jupiter
have been configured.
--
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]
