dombizita commented on PR #3988: URL: https://github.com/apache/ozone/pull/3988#issuecomment-1358575497
> ClusterSync terminology is better as we have few other tasks and sub-tasks also which do processing on data synced from OM and SCM DB both on a periodic basis and few internal tasks for SCM container cache sync (#3947 ) because of that I think we should keep `OM sync`, as we kind of have `SCM sync` and that is not happening with this button. also the `last refreshed` is not correct regarding SCM, only the OM DB related data is refreshed. I still don't think this rename is useful, as users will believe all the data related to the cluster is refreshed at a time and they can do that with the button, but that is not true. it is better to have something that users can look up to and search for its meaning, than misleading them. one way is to change it to `DB sync` and later we can add one for SCM too as it is advised [here](https://github.com/apache/ozone/pull/3929#issuecomment-1314287362). @myskov any thoughts on this? -- 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]
