priyeshkaratha commented on PR #10767:
URL: https://github.com/apache/ozone/pull/10767#issuecomment-5054460876

   > @priyeshkaratha , changes largely LGTM, just a point worth noting that, it 
is always better to point the default to the first available DN and only fall 
back to the error card when every DN is unavailable, else user always lands on 
an error card on first load and must manually switch — even though a healthy DN 
exists.
   
   Thanks @devmadhuu. This make sense. I have updated PR which will do auto 
switch and if all of the DNs are down it will show error card.


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