> On June 12, 2017, 10:48 a.m., Attila Doroszlai wrote: > > utility/src/main/java/org/apache/ambari/swagger/AmbariSwaggerReader.java > > Lines 56 (patched) > > <https://reviews.apache.org/r/59929/diff/1/?file=1744990#file1744990line56> > > > > Should be `LOG`, not `logger`
Unfortunately, LOG denotes an existing member variable which is a (less sophisticated than slfj) Maven logger. I chose slf4j for logging, but I had to chose a different variable name. > On June 12, 2017, 10:48 a.m., Attila Doroszlai wrote: > > utility/src/main/java/org/apache/ambari/swagger/AmbariSwaggerReader.java > > Lines 80 (patched) > > <https://reviews.apache.org/r/59929/diff/1/?file=1744990#file1744990line80> > > > > Please avoid + in `debug` message, use placeholders (`{}`) instead. This is a reainder of switching logger implementations back and forth (slf4j vs maven logging). I'll fix it. - Balázs Bence ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59929/#review177602 ----------------------------------------------------------- On June 12, 2017, 10:09 p.m., Balázs Bence Sári wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59929/ > ----------------------------------------------------------- > > (Updated June 12, 2017, 10:09 p.m.) > > > Review request for Ambari, Attila Doroszlai, Jaimin Jetly, Oliver Szabo, > Sumit Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-21207 > https://issues.apache.org/jira/browse/AMBARI-21207 > > > Repository: ambari > > > Description > ------- > > Wrote a custom SwaggerReader in the utility project that handles nested > API's. There is currently one nested API which is annotated for swagger: > org.apache.ambari.server.api.services.ServiceService (nested under > ClusterService). Using the custom reader operation paths and parameters are > generated correctly for the nested API. > > > Diffs > ----- > > ambari-project/pom.xml 64d1416 > ambari-server/pom.xml 669f90f > > ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java > f61fb2a > > ambari-server/src/main/java/org/apache/ambari/server/api/services/ServiceService.java > a28c4aa > utility/pom.xml 7d5eb93 > utility/src/main/java/org/apache/ambari/swagger/AmbariSwaggerReader.java > PRE-CREATION > > utility/src/test/java/org/apache/ambari/swagger/AmbariSwaggerReaderTest.java > PRE-CREATION > utility/src/test/resources/log4j.properties PRE-CREATION > > > Diff: https://reviews.apache.org/r/59929/diff/2/ > > > Testing > ------- > > Tested Swagger API generation manually. Wrote new unit tests. Run all unit > tests for the ambari-server and utility projects, no errors or failures. > > > Thanks, > > Balázs Bence Sári > >
