Re: [PR] HDDS-11023. Recon Disk Usage null conditions not handled properly for null response [ozone]

[via GitHub]

smitajoshi12 commented on PR #6894:
URL: https://github.com/apache/ozone/pull/6894#issuecomment-2225362381

   > Thanks for working on this patch, @smitajoshi12 I appreciate you catching 
this issue and implementing the necessary changes. However, I believe we can 
present the information displayed to the user about Namespace initialization in 
a much clearer way, as the Namespace can be in initialization mode for two 
specific reasons:
   > 
   > 1. The metadata is too large and is taking time to initialize.
   > 2. The OM snapshot has not yet been fetched from the OM.
   > 
   > It's important for the user to know that there is no problem with the 
data; rather, there is just a delay in building the NSSummary. Instead of 
having the user click the "Show Metadata for Current Path" button, we should 
display a pop-up message. This message should not be an alert but a simple 
notification indicating that the state is initializing and that they need to 
wait.
   > 
   > Additionally, the error message shown in the screenshot can be improved to 
something more user-friendly like:
   > 
   > "**_Metadata Initialization: The metadata is currently initializing. 
Please wait a moment and try again later._**”
   > 
   > <img alt="image" width="425" 
src="https://private-user-images.githubusercontent.com/98023601/348139071-3d9fea4d-691d-4f3a-a046-0ff716c8de03.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3ODMyODEsIm5iZiI6MTcyMDc4Mjk4MSwicGF0aCI6Ii85ODAyMzYwMS8zNDgxMzkwNzEtM2Q5ZmVhNGQtNjkxZC00ZjNhLWEwNDYtMGZmNzE2YzhkZTAzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzEyVDExMTYyMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWIyOTkyYjhlNzFkYzQ3M2JhNDg4MzRkYTFmMWQ4MjRlOGIwOTFhMDFjMDRlZTgwMTVlNDNlODMwZTRjYTEwZDEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.YQWe7UyBVcHn_YlDZkYJv51EhfmdWWWZmybwCpGJ2VM";>
   
   @ArafatKhan2198 
   Completed  Changes suggested by you attached screenshot with Info 
Notification


-- 
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-unsubscr...@ozone.apache.org

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


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