New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FLINK-9045][REST] Add logging message for web UI address #5814
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me. +1 for merging.
Additionally, we could try to start the WebMonitorEndpoint
with port 8081
if the user calls createLocalEnvironmentWithWebUI
on the StreamExecutionEnvironment
. The legacy FlinkMiniCluster
also started the WebRuntimeMonitor
with port 8081
.
I have removed a section from the This should cause the UI to run at |
Correction: I had to modify the |
Looks good to merge! 👍 |
…endly logging message for web UI address - add back known logging mesages about webUI address - do not set random port in local stream environment This closes apache#5814.
…endly logging message for web UI address - add back known logging mesages about webUI address - do not set random port in local stream environment This closes apache#5814.
…endly logging message for web UI address -add back known logging mesages about webUI address -do not set random port in local stream environment This closes apache#5814.
…endly logging message for web UI address -add back known logging mesages about webUI address -do not set random port in local stream environment This closes apache#5814.
…endly logging message for web UI address -add back known logging mesages about webUI address -do not set random port in local stream environment This closes apache#5814.
…endly logging message for web UI address -add back known logging mesages about webUI address -do not set random port in local stream environment This closes #5814.
…endly logging message for web UI address -add back known logging mesages about webUI address -do not set random port in local stream environment This closes apache#5814.
What is the purpose of the change
Adds a logging message explicitly saying on which address and port the web UI is listening on.
This message is logged in the legacy code-base, but with flip6 we were only logging the REST server address. While technically the same it may not be obvious to users that there is a difference.
Verifying this change
Not tested.