henrybear327 commented on code in PR #10760:
URL: https://github.com/apache/ozone/pull/10760#discussion_r4245658926
##########
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/local/LocalOzoneCluster.java:
##########
@@ -337,6 +365,23 @@ InetSocketAddress getS3gBoundAddress() {
return s3Gateway != null ? s3Gateway.getHttpAddress() : null;
}
+ @Override
+ public int getReconPort() {
+ if (!config.isReconEnabled()) {
+ return -1;
+ }
+ return preparedConfiguration == null ? config.getReconPort()
+ : preparedConfiguration.getReconPort();
+ }
+
+ @Override
+ public String getReconEndpoint() {
+ if (!config.isReconEnabled()) {
+ return "";
+ }
+ return "http://"; + getDisplayHost() + ":" + getReconPort();
Review Comment:
@smengcl Thanks for the patch and I have applied it.
@jojochuang I kept the advertised endpoints on HTTP. Should I file a
follow-up Jira ticket to finish up implementing the HTTPS part?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
