XbaoWu commented on PR #6616:
URL: https://github.com/apache/hadoop/pull/6616#issuecomment-2011278871
> @XbaoWu - could you please address the 3 checkstyle violations generated
by your patch?
>
> TestCheckRemoveZKNodeRMStateStore.java:95: TestZKRMStateStoreInternal
store;:32: Variable 'store' must be private and have accessor methods.
[VisibilityModifier]
>
> TestCheckRemoveZKNodeRMStateStore.java:96: String workingZnode;:12:
Variable 'workingZnode' must be private and have accessor methods.
[VisibilityModifier]
>
> TestCheckRemoveZKNodeRMStateStore.java:366: public void
testTransitionedToStandbyAfterCheckNode(RMStateStoreHelper stateStoreHelper)
throws Exception {: Line is longer than 100 characters (found 109). [LineLength]
Okay, I 've solved these non-standard code.
--
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]
