wuchong commented on issue #2272: URL: https://github.com/apache/fluss/issues/2272#issuecomment-3695919942
@gyang94 I understand your point. However, introducing a new config like `bind.listeners.port.override` would unnecessarily bloat the configuration space and create confusion with the existing `bind.listeners` setting. This issue is specific to `local-cluster.sh`. There’s no broader need for such a configuration. It’s better to handle it in the `local-cluster.sh` script, rather than exposing complexity to all users. Moreover, `bind.listeners.port.override` wouldn’t cleanly solve the problem: if a user configures multiple listeners like `bind.listeners=INTERNAL://10.0.0.23:9123,EXTERNAL://10.0.0.23:9124`, hard-coding `-Dbind.listeners.port.override=FLUSS:0` would fail because the TabletServer does not use the same listener name. I think the solution is very simple: 1. Parse the `bind.listeners` value from `server.yaml`, 2. Replace the port in **each** listener URI with `0`, 3. Pass the updated value via `-Dbind.listeners=...` when launching the TabletServer. This keeps the logic contained, avoids config sprawl, and works for any listener setup. -- 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]
