----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58356/#review171877 -----------------------------------------------------------
utility/src/main/java/org/apache/ambari/checkstyle/UndocumentedRestApiOperationCheck.java Lines 36 (patched) <https://reviews.apache.org/r/58356/#comment244875> You could use here ImmutableSet.of() - Sebastian Toader On April 13, 2017, 3:48 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58356/ > ----------------------------------------------------------- > > (Updated April 13, 2017, 3:48 p.m.) > > > Review request for Ambari, Balázs Bence Sári, Jaimin Jetly, Jayush Luniya, > Oliver Szabo, Sumit Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-20735 > https://issues.apache.org/jira/browse/AMBARI-20735 > > > Repository: ambari > > > Description > ------- > > Created checkstyle check to ensure either @ApiOperation or @ApiIgnore is > present on all REST API methods > > > Diffs > ----- > > ambari-server/checkstyle.xml bf7698dddd760ac8ddba1d692bcd448c19d939dc > ambari-server/pom.xml fbc2b9fd129dca0828624fe76e33616b6919f429 > ambari-server/src/main/assemblies/server.xml > 27835265f78a3a7aed37eaa544e25679eee8b400 > utility/src/main/java/org/apache/ambari/annotations/ApiIgnore.java > PRE-CREATION > > utility/src/main/java/org/apache/ambari/checkstyle/UndocumentedRestApiOperationCheck.java > PRE-CREATION > > utility/src/test/java/org/apache/ambari/checkstyle/UndocumentedRestApiOperationCheckTest.java > PRE-CREATION > > utility/src/test/resources/org/apache/ambari/checkstyle/InputRestApiOperation.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/58356/diff/1/ > > > Testing > ------- > > Includes unit test for the new check. > Also ran the check on ambari-server: produces 315 errors. > > > Thanks, > > Attila Doroszlai > >
