fapifta commented on code in PR #6725:
URL: https://github.com/apache/ozone/pull/6725#discussion_r1633187492
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/MiniOzoneHAClusterImpl.java:
##########
@@ -661,7 +661,6 @@ private void initSCMHAConfig() {
conf.set(scmHttpAddrKey, localhostWithFreePort());
conf.set(scmHttpsAddrKey, localhostWithFreePort());
conf.set(scmSecurityAddrKey, localhostWithFreePort());
- conf.set("ozone.scm.update.service.port", "0");
Review Comment:
Hi @kerneltime,
yes it is related, the CRL distribution mechanism uses the SCMUpdateService
(actually the CRL distribution is the only thing uses it), and it has a config
object UpdateServiceConfig which defines this config.
As in this PR I am removing the update service as it is not used by anything
else, it is reasonable to remove the config option and its usages also.
--
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]
