adoroszlai commented on PR #4332:
URL: https://github.com/apache/ozone/pull/4332#issuecomment-1456344885

   > Then, how about adding a list of known tags as a comment in 
ozone-default.xml like this?
   
   I think it would be more useful to:
   
   1. Update `hadoop.tags.custom` and `ozone.tags.system` to reflect the list 
of tags actually in use (combined from XML and annotated classes, i.e. 
[used.txt](https://github.com/apache/ozone/files/10899778/used.txt)).  My 
understanding is that the both properties should have the same value.
   2. Remove the list of tags from the comment, and provide some instructions 
instead, something like:
       ```
       * List of existing tags is defined by the properties 
`hadoop.tags.custom` and `ozone.tags.system`.
       * If you start using a new tag, please update both lists.
       ````


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