[GitHub] [ozone] elek commented on pull request #1600: HDDS-4475.Extend DatanodeChunkGenerator to write all on all pipelines…

[GitBox <gi...@apache.org>]

elek commented on pull request #1600:
URL: https://github.com/apache/ozone/pull/1600#issuecomment-737122197


   Thanks to explain it, now I got it. 
   
   In that case you should have additional modification to use the current loop:
   
   You should either select the first one OR the pipelines assigned to the datanode.
   
   Change `private XceiverClientSpi xceiverClientSpi` to `private List<XceiverClientSpi> clients` to have clients to all pipelines.
   
   And in `writeChunk` select a random one instead of using the single one. 
   
   (You should also initialize the full list of clients and close them...) 
   
   Also: how would you like to guarantee that if the selected datanode is the leader of bot pipelines?
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org