adoroszlai opened a new pull request #22: [AMBARI-18361] Avoid unnecessary 
compilation
URL: https://github.com/apache/ambari-infra/pull/22
 
 
   ## What changes were proposed in this pull request?
   
   Fix for:
   
   1. all `ambari-infra-manager` classes are recompiled even if none has changed
   2. all classes are recompiled in each submodule that has any changed java 
source
   
   ```
   $ mvn clean test-compile
   $ mvn test-compile
   ...
   [INFO] --- maven-compiler-plugin:3.0:compile (default-compile) @ 
ambari-infra-manager ---
   [INFO] Changes detected - recompiling the module!
   [INFO] Compiling 100 source files to ambari-infra-manager/target/classes
   ...
   $ touch 
ambari-infra-manager/src/main/java/org/apache/ambari/infra/conf/InfraManagerDataConfig.java
   $ mvn test-compile
   ...
   [INFO] --- maven-compiler-plugin:3.0:compile (default-compile) @ 
ambari-infra-manager ---
   [INFO] Changes detected - recompiling the module!
   [INFO] Compiling 100 source files to ambari-infra-manager/target/classes
   ```
   
   ## How was this patch tested?
   
   ```
   $ mvn clean test-compile
   $ mvn test-compile
   ...
   [INFO] --- maven-compiler-plugin:3.3:compile (default-compile) @ 
ambari-infra-manager ---
   [INFO] Nothing to compile - all classes are up to date
   ...
   $ touch 
ambari-infra-manager/src/main/java/org/apache/ambari/infra/conf/InfraManagerDataConfig.java
   $ mvn test-compile
   ...
   [INFO] --- maven-compiler-plugin:3.3:compile (default-compile) @ 
ambari-infra-manager ---
   [INFO] Compiling 1 source file to ambari-infra-manager/target/classes
   ```
   
   ```
   $ mvn clean test
   ...
   [INFO] BUILD SUCCESS
   $ mvn test
   ...
   [INFO] BUILD SUCCESS
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to