sonam-vend commented on a change in pull request #12938:
URL: https://github.com/apache/beam/pull/12938#discussion_r550450329



##########
File path: runners/flink/1.10/job-server/build.gradle
##########
@@ -29,3 +29,7 @@ project.ext {
 
 // Load the main build script which contains all build logic.
 apply from: "$basePath/flink_job_server.gradle"
+
+dependencies {
+  permitUnusedDeclared project(":runners:flink:1.10")

Review comment:
       `:runners:flink:1.10:job-server:analyzeClassesDependencies `task fails 
if we don't add `permitUnusedDeclared project(":runners:flink:1.10")` 
   
   ```
   Task :runners:flink:1.10:job-server:analyzeClassesDependencies FAILED
   
   FAILURE: Build failed with an exception.
   
   * What went wrong:
   Execution failed for task 
':runners:flink:1.10:job-server:analyzeClassesDependencies'.
   > Dependency analysis found issues.
     unusedDeclaredArtifacts: 
      - beam.runners.flink:1.10:2.28.0-SNAPSHOT@jar
   ```
   
   As it is transitively being declared, but we do not need it here. Thats why 
it is important to declare it permitUnsedDeclared.
   
   @ibzib 




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


Reply via email to