Hexiaoqiao commented on pull request #3284: URL: https://github.com/apache/hadoop/pull/3284#issuecomment-898297579
Thanks @LeonGao91 and @fengnanli . It is great improvement here. Just some nits here, a. We should put the new configuration item to hdfs-default.xml. b. If enable this feature, DNS Resolver has to support resolve multiple hostnames from one domain name, right? Just suggest add some note to HDFSHighAvailabilityWithQJM.md. Thanks. -- 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]
