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


   ### What changes were proposed in this pull request and why?
   Exclude drivers for Oracle, MySQL, and MariaDB to avoid licensing violations.
   
   Misc:
   1. Group all JDBC drivers in metastore-server pom.xml together for
   readability.
   2. Add MariaDB driver in metastore-server pom.xml for keeping things
   uniform.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Build binary distribution and manually inspect lib directory.
   
   ```
   mvn clean package -DskipTests -Pdist
   ls 
packaging/target/apache-hive-4.0.0-SNAPSHOT-bin/apache-hive-4.0.0-SNAPSHOT-bin/lib/
 | grep -e "postgres" -e "ojdbc" -e "mssql" -e "mariadb" -e "derby" -e "mysql"
   ```
   Do the same for metastore:
   ```
   ls -l apache-hive-metastore-4.0.0-SNAPSHOT-bin/lib/ | grep -e "postgres" -e 
"ojdbc" -e "mssql" -e "mariadb" -e "derby" -e "mysql"
   -rw-r--r-- 1 stamatis stamatis  3232158 Jan 22  2020 derby-10.14.1.0.jar
   -rw-r--r-- 1 stamatis stamatis   792491 Jan 22  2020 
mssql-jdbc-6.2.1.jre8.jar
   -rw-r--r-- 1 stamatis stamatis   932808 Jan 22  2020 postgresql-42.2.14.jar
   ```


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