[GitHub] [ozone] fapifta commented on pull request #2979: HDDS-5954. EC: Review the TODOs in GRPC Xceiver client and fix them.

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

fapifta commented on pull request #2979:
URL: https://github.com/apache/ozone/pull/2979#issuecomment-1020111002


   Hi @umamaheswararao,
   
   thank you for the reviews, I will incorporate these comments into the version I will post under the new JIRA tickets, as I separated out 4 different things that we need to do in order to get through this task.
   1. add the documentation about the learning points on master as that contains useful knowledge, and remove the TODO-s and related comments from the client code. (HDDS-6217)
   2. deprecate the standalone client for writes (HDDS-6218)
   3. remove the standalone client usage from tests where we still have it (HDDS-6219)
   4. add a alternate client for EC writes that is only used by EC, and change the write synchronization as it is needed for EC. (HDDS-6220)
   
   With that I am closing this PR, and will post the new ones later today, as I am moving out the different pieces to different branches.


-- 
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: issues-unsubscribe@ozone.apache.org

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