smitajoshi12 commented on PR #4940: URL: https://github.com/apache/ozone/pull/4940#issuecomment-1609361940
> @smitajoshi12 Thanks for working on this patch, Few comments: > > 1. Container Mismatch Info screenshot is not inline with what is provided in PR description. > 2. In all the pages, "keys" column header label should be "Count Of Keys". > 3. Container Mismatch Info page data showing key count as 1, however on expansion of respective container id shows multiple keys, so looks like pages based on test data in db.json is not correct. > 4. Key value shown on OpenKeys page doesn't seem correct. > 5. Path shown in "Keys Pending For Deletion" seems not correct. > 6. "Keys" label in table of page "Deleted Container Keys" should be "Count Of Keys". @devmadhuu Completed Changes and added latest screenshots. -- 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]
