zabetak opened a new pull request, #4317: URL: https://github.com/apache/hive/pull/4317
### What changes were proposed in this pull request? 1. Remove optional from `dependencyManagement` section since it has no effect (MNG-5227) 2. Add explicitly the optional tag in every declaration of a JDBC driver. 3. Add derby dependency in `hcatalog-core` module cause it is needed to compile `DerbyPolicy` class in tests; it was not necessary before cause the dependency was transitively inherited. 4. Remove (now) redundant JDBC driver exclusions from packaging module (`bin.xml`). ### Why are the changes needed? 1. Stop JDBC drivers propagation to other Hive modules and 3rd-party projects 2. Ensure license compliance for those drivers that are in Category-X (https://www.apache.org/legal/resolved.html#optional) ### Does this PR introduce _any_ user-facing change? * The users will need to download the appropriate driver explicitly. * Dependent projects should add an explicit dependency if they need to use a specific DBMS. ### How was this patch tested? * Compared the results of `mvn dependency:tree` before and after the changes. * Verified that JDBC driver dependencies appear only in the expected places where they are declared. -- 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]
