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

   ### What changes were proposed in this pull request?
   1. Use testcontainers library to manage the lifecycle of dockerized databases
   2. Use the newest version of the library (1.21.3) everywhere
   
   ### Why are the changes needed?
   1. Reduce maintenance burden since we no longer have to manually craft and 
fix docker handling logic
   2. Stabilize CI and by avoiding common problems as port conflicts, container 
leaks, etc., which are handled automatically by the library
   3.  Enhance OS independence since we no longer call Runtime.exec methods. 
The underlying docker-java library takes care of using the appropriate APIs 
(REST, etc.)
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   ```
   mvn test -Dtest=TestMssql,TestDerby,TestMysql,TestOracle,TestPostgres 
-Dtest.groups= -Dverbose.schematool
   ```


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