tomscut commented on pull request #3564:
URL: https://github.com/apache/hadoop/pull/3564#issuecomment-952467805


   > ProcessReport from dead or unregistered node
   
   Thanks @tasanuma for your comment and pointing out this issue.
   
   When the instantiation of DatanodeID is DatanodeRegistration, it overrides 
```toString()```, so the output is a lot, but the address is ```IP:port```.
   
   When the nodeID instantiation object is a DatanodeDescriptor, it doesn't 
override the ```toString()``` method, so the output is very little, just the 
address ```IP:port```.
   
   I intended to print ```hostname:port``` as @aajisaka suggested. But there's 
something wrong with my implementation. I'll leave the DN address format as it 
was: ```IP:port``` (because changing the ```toString()``` of the datanodeID or 
DatanodeRegistration might cause other compatibility issues).  What do you 
think of this way? Thank you.


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