wanglijie95 commented on a change in pull request #18023:
URL: https://github.com/apache/flink/pull/18023#discussion_r776757395
##########
File path:
flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/DefaultVertexParallelismInfoTest.java
##########
@@ -41,15 +41,15 @@ public void parallelismInvalid() {
assertThrows(
"parallelism is not in valid bounds",
IllegalArgumentException.class,
- () -> new DefaultVertexParallelismInfo(-1, 1, ALWAYS_VALID));
+ () -> new DefaultVertexParallelismInfo(-2, 1, ALWAYS_VALID));
}
@Test
public void maxParallelismInvalid() {
assertThrows(
"max parallelism is not in valid bounds",
IllegalArgumentException.class,
- () -> new DefaultVertexParallelismInfo(1, -1, ALWAYS_VALID));
+ () -> new DefaultVertexParallelismInfo(1, -2, ALWAYS_VALID));
}
@Test
Review comment:
I will add tests.
--
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]
