[GitHub] [beam] ibzib commented on a change in pull request #14053: [BEAM-10961] enable strict dependency checking for sdks/java/io/hcatalog

[GitBox <gi...@apache.org>]

ibzib commented on a change in pull request #14053:
URL: https://github.com/apache/beam/pull/14053#discussion_r581580461



##########
File path: sdks/java/io/hcatalog/build.gradle
##########
@@ -65,8 +66,6 @@ dependencies {
   provided library.java.hadoop_common
   provided "org.apache.hive:hive-exec:$hive_version"
   provided(group: "org.apache.hive.hcatalog", name: "hive-hcatalog-core", version: hive_version) {
-    exclude group: "org.apache.hive", module: "hive-exec"

Review comment:
       Why unexclude these? They don't seem to cause a problem with the plugin?
   
   ```
   * What went wrong:
   Execution failed for task ':sdks:java:io:hcatalog:analyzeClassesDependencies'.
   > Dependency analysis found issues.
     usedUndeclaredArtifacts:
      - joda-time:joda-time:2.10.10@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.

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