JacksonYao287 commented on pull request #2435:
URL: https://github.com/apache/ozone/pull/2435#issuecomment-887240050


   thanks @ChenSammi for the review! 
   >Precondition check that
   >after DN is removed from topology, DatanodeDetails instance returned from 
nodeStateManager has no parent.
   >after DN is added back into topology, DatanodeDetails instance returned 
from nodeStateManager has parent correctly set.
   >And would you add a new UT for it.
   
   i  will make this change


-- 
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]

Reply via email to