ZanderXu commented on PR #4628: URL: https://github.com/apache/hadoop/pull/4628#issuecomment-1209185985
@ferhui Master, thanks for your helping review. > Change visibility because of Test cases, right? Is there a way to avoid changing it and make it concise? Yes, change the visibility is only for UT. The related classes are in different packages, and I need to write some edits to JN and verify them after HA, so it's difficult to code the UT without changing the visibility. Do you have some good ideas? -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
