LuciferYang opened a new pull request #29103:
URL: https://github.com/apache/spark/pull/29103


   ### What changes were proposed in this pull request?
   
   Add additional configuration to `maven-clean-plugin` to ensure cleanup 
`metastore_db` and `spark-warehouse` directory when execute `mvn clean` command 
to make sure `mvn clean test` successful.
   
   ### Why are the changes needed?
   Now we support two version of build-in hive and there are some test 
generated data not in target dir like `spark-warehouse`, `metastore_db`,  they 
don't clean up automatically when we run `mvn clean` command. 
   
   So f we run `mvn clean test -pl sql/hive -am -Phadoop-2.7 -Phive -Phive-1.2 
` , the `metastore_db` dir will created and meta data will remains after test 
complete.
   
   Then we need manual cleanup `metastore_db` directory to ensure `mvn clean 
test -pl sql/hive -am -Phadoop-2.7 -Phive` command can succeed because the 
residual metastore data is not compatible.
   
   `spark-warehouse` will also cause test failure in some data residual 
scenarios too.
   
   This pr is used to simplify manual cleanup `metastore_db` and 
`spark-warehouse` directory operation.
   
   ### How was this patch tested?
   
   Manual execute `mvn clean test -pl sql/hive -am -Phadoop-2.7 -Phive 
-Phive-1.2`, then execute `mvn clean test -pl sql/hive -am -Phadoop-2.7 
-Phive`, both commands should succeed.
   


----------------------------------------------------------------
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.

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



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

Reply via email to