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

   ## What changes were proposed in this pull request?
   
   Problem: `(hdds|ozone)-version-info.properties` include MD5 checksum of all 
prod Java and protobuf source files under `hadoop-(hdds|ozone)`.  They are then 
added to `(hdds|ozone)-common.jar`, which causes build cache miss for all 
modules that depend on them.  Example: 499bbd3e3ce32fe715acf9823ef24b676977b225 
only changed Recon, which `ozone-integration-test` does not depend on.  But 
integration tests encountered [cache 
miss|https://develocity.apache.org/c/wz5yoeciqmxme/ckv2d3t3s52bq/goal-inputs?expanded=WyJ0bTRxNHFvbmxyaTJhLXRlc3RjbGFzc3BhdGhlbGVtZW50cyJd]
 due to the difference in `ozone-common.jar`.
   
   This change removes `srcChecksum` and related methods.  `<source>` input is 
mandatory for `hadoop-maven-plugins`, so it is kept with a smaller input 
directory (to reduce file scanning time).
   
   https://issues.apache.org/jira/browse/HDDS-17538
   
   ## How was this patch tested?
   
   Verified `srcChecksum` no longer present in the properties files, nor in 
`ozone version` output.
   
   ```
   $ ozone version
   ...
   Source code repository [email protected]:apache/ozone.git -r 
7d90b5a22dd2cce60555414a6e5dafeec41ebbe4
   ...
   ```
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/40737447008
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to