zabetak opened a new pull request, #5826:
URL: https://github.com/apache/hive/pull/5826

   ### Why are the changes needed?
   Previously the maven artifacts were stored under .git/.m2 directory. In 
terms of functionality the exact location (.git/.m2 vs .m2) does not matter. 
However, putting the artifacts under the .git folder can lead into confusion 
and false conclusions for people not very familiar with the specificities of 
the CI.
   
   For instance, someone who is not gonna look into detail on what is present 
under the .git directory may think that the .git data for the project are huge. 
Using commands such as `du -h --max-depth=1`, which is also used in some places 
by the CI, can endorse such false impressions.
   
   Moreover, the .m2 directory is a more common place for storing the maven 
artifacts in CI/CD pipelines.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   * Ensure tests run fine
   * Inspect CI logs especially those related to `du -h --max-depth=1` command


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to