-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60117/
-----------------------------------------------------------
(Updated June 15, 2017, 1:52 p.m.)
Review request for Ambari, Attila Doroszlai, Alejandro Fernandez, Jonathan
Hurley, Robert Levas, and Sebastian Toader.
Changes
-------
rebase patch -> move new files
Bugs: AMBARI-21257
https://issues.apache.org/jira/browse/AMBARI-21257
Repository: ambari
Description
-------
- use org.apache.ambari groups (makes sense to use the same group with ambari
modules)
- in this apache project, the convention is to use ambari prefix for ambari
modules (can be without it as well, see e.g.: apache spark), to not mix both
method, add ambari prefix to utility and serviceadvisor modules
Diffs (updated)
-----
ambari-agent/pom.xml 7690fcf
ambari-infra/pom.xml a5a5b38
ambari-logsearch/pom.xml 6f78abe
ambari-metrics/ambari-metrics-timelineservice/pom.xml 3419f16
ambari-metrics/pom.xml 02f9574
ambari-project/pom.xml b06bd18
ambari-server/pom.xml ca3b2da
pom.xml 4e27ca6
serviceadvisor/pom.xml ecf6d8b
serviceadvisor/src/main/java/org/apache/ambari/serviceadvisor/ServiceAdvisor.java
serviceadvisor/src/main/java/org/apache/ambari/serviceadvisor/ServiceAdvisorCommandType.java
utility/checkstyle.xml
utility/pom.xml ac91474
utility/src/main/java/category/AlertTest.java
utility/src/main/java/category/AmbariUpgradeTest.java
utility/src/main/java/category/BlueprintTest.java
utility/src/main/java/category/FastTest.java
utility/src/main/java/category/KerberosTest.java
utility/src/main/java/category/MetricsTest.java
utility/src/main/java/category/SlowTest.java
utility/src/main/java/category/StackUpgradeTest.java
utility/src/main/java/org/apache/ambari/annotations/ApiIgnore.java
utility/src/main/java/org/apache/ambari/checkstyle/AvoidTransactionalOnPrivateMethodsCheck.java
utility/src/main/java/org/apache/ambari/checkstyle/UndocumentedRestApiOperationCheck.java
utility/src/main/java/org/apache/ambari/swagger/AmbariSwaggerReader.java
utility/src/main/resources/checkstyle_packages.xml
utility/src/test/java/org/apache/ambari/checkstyle/AvoidTransactionalOnPrivateMethodsCheckTest.java
utility/src/test/java/org/apache/ambari/checkstyle/UndocumentedRestApiOperationCheckTest.java
utility/src/test/java/org/apache/ambari/swagger/AmbariSwaggerReaderTest.java
utility/src/test/resources/log4j.properties
utility/src/test/resources/org/apache/ambari/checkstyle/InputRestApiOperation.java
utility/src/test/resources/org/apache/ambari/checkstyle/InputTransactionalOnPrivateMethods.java
Diff: https://reviews.apache.org/r/60117/diff/2/
Changes: https://reviews.apache.org/r/60117/diff/1-2/
Testing
-------
done. modules could find each other
Thanks,
Oliver Szabo
