fateh288 commented on PR #6677: URL: https://github.com/apache/hadoop/pull/6677#issuecomment-2024377833
Style error #1: -nonSuperUser is protected and is in-line with other similar member variables of the class. Style error #2: not related to PR. Only whitespace removed from line for style check Style error #3,#4,#5: not related to PR. Only whitespace removed from line for style check Style error #6: Line is longer than 100 characters -- can be fixed if really necessary, else can be followed up in another future PR. @dineshchitlangia do you suggest fixing this now (somehow the dev-support/bin/test-patch does not pick up any of these style errors which the CI pipeline is throwing here in the PR)? Unit test failures: not related to the patch -- 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