corgy-w commented on code in PR #7903:
URL: https://github.com/apache/seatunnel/pull/7903#discussion_r1816365003
##########
seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base/src/test/java/org/apache/seatunnel/engine/e2e/RestApiIT.java:
##########
@@ -105,7 +105,8 @@ void beforeClass() throws Exception {
node2Tags.setAttribute("node", "node2");
Config node2hzconfig =
node1Config.getHazelcastConfig().setMemberAttributeConfig(node2Tags);
node2Config = ConfigProvider.locateAndGetSeaTunnelConfig();
- node2Config.getEngineConfig().getHttpConfig().setPort(8081);
+ // Dynamically generated port
+
node2Config.getEngineConfig().getHttpConfig().setEnableDynamicPort(true);
Review Comment:
> Can we provide a port range? Otherwise, how do I know which port is bound?
Get, I will add a configuration control to the cumulative range, like
'port-count', Res: 8080-8180, What do you think
--
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]
