bshashikant edited a comment on pull request #1068:
URL: https://github.com/apache/hadoop-ozone/pull/1068#issuecomment-643291103


   Thanks @lokeshj1703 for working on this. The change looks ok. 
   I think in this case, ozone client has already exhausted the max attempt 
count trying to write the data but eventually failed .
   
   How about this case, where the client attempts to write and in turn run into 
ContainerNot open exception repeatedly? In such cases, ozone client may keep on 
retrying for considerable amount of time which i think we should not do.
   
   May be its ok to fail irrespective of exception after trying for maximum 
attempts.
   Also, the test failure seems to be related. Can u plz check as well?


----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to