adoroszlai opened a new pull request, #7196:
URL: https://github.com/apache/ozone/pull/7196

   ## What changes were proposed in this pull request?
   
   Checkstyle applies some of its checks to properties files in 
`src/.../resources` directory.  However, checkstyle is run for PRs only if some 
Java files are changed (or checkstyle config itself).  Thus CI may fail to 
catch some checkstyle violation in properties files.
   
   https://issues.apache.org/jira/browse/HDDS-11458
   
   ## How was this patch tested?
   
   Added test case for selective checks using the following commit:
   
   ```
   commit 71b8bdd8becf72d6f7d4e7986895504b8259b3e5
   ...
   
       HDDS-4794. No appenders could be found for logger - in tests (#1896)
   
    hadoop-hdds/client/src/test/resources/log4j.properties            |  2 +-
    hadoop-hdds/common/src/test/resources/log4j.properties            |  2 +-
    hadoop-hdds/container-service/src/test/resources/log4j.properties |  2 +-
    hadoop-hdds/framework/src/test/resources/log4j.properties         | 23 
+++++++++++++++++++++++
    hadoop-ozone/insight/src/test/resources/log4j.properties          | 23 
+++++++++++++++++++++++
    hadoop-ozone/tools/src/test/resources/log4j.properties            | 23 
+++++++++++++++++++++++
    6 files changed, 72 insertions(+), 3 deletions(-)
   ```


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

Reply via email to