zentol commented on code in PR #19523:
URL: https://github.com/apache/flink/pull/19523#discussion_r854032674
##########
flink-runtime/src/main/java/org/apache/flink/runtime/minicluster/MiniClusterConfiguration.java:
##########
@@ -206,6 +207,11 @@ public Builder setHaServices(MiniCluster.HaServices
haServices) {
return this;
}
+ public Builder withRandomPorts() {
Review Comment:
The MiniCluster is in a weird spot where it is not a pure testing utility.
If you consider it to just be a single JVM Flink deployment, then it makes
perfect sense for it to use the usual ports.
I'm wary of changing the default MiniCluster behavior because it may break
plenty of stuff for users.
--
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]
