zhengchenyu commented on PR #897:
URL: 
https://github.com/apache/incubator-uniffle/pull/897#issuecomment-1558935138

   > > > > ## Appendix
   > > > > In this PR, when RequireUpperBoundDeps is enable, some error were 
thrown. Two error:
   > > > > 
   > > > > * The version of commons-codec:commons-codec is lower than 
hadoop3.2's version.
   > > > >   I update commons-codec.version to 1.11 in hadoop3.2 profile.
   > > > > * The other error has nothing to do with uniffle, 
RequireUpperBoundDeps will not pass in hadoop-3.2 profile. So I skip 
RequireUpperBoundDeps in hadoop-3.2 profile.
   > > > > 
   > > > > Errors are described as below:
   > > > > ```
   > > > > Failed while enforcing RequireUpperBoundDeps. The error(s) are [
   > > > > Require upper bound dependencies error for 
commons-codec:commons-codec:1.9 paths to dependency are:
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-commons-codec:commons-codec:1.9
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-org.apache.httpcomponents:httpclient:4.5.6
   > > > >       +-commons-codec:commons-codec:1.9 (managed) <-- 
commons-codec:commons-codec:1.11
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-org.apache.hadoop:hadoop-auth:3.2.1
   > > > >       +-commons-codec:commons-codec:1.9 (managed) <-- 
commons-codec:commons-codec:1.11
   > > > > ,
   > > > > Require upper bound dependencies error for 
org.codehaus.jackson:jackson-core-asl:1.9.2 paths to dependency are:
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-core-asl:1.9.2
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-org.apache.avro:avro:1.7.7
   > > > >       +-org.codehaus.jackson:jackson-core-asl:1.9.13
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-mapper-asl:1.9.2
   > > > >         +-org.codehaus.jackson:jackson-core-asl:1.9.2
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-jaxrs:1.9.13 (managed) <-- 
org.codehaus.jackson:jackson-jaxrs:1.9.2
   > > > >         +-org.codehaus.jackson:jackson-core-asl:1.9.13
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-xc:1.9.13 (managed) <-- 
org.codehaus.jackson:jackson-xc:1.9.2
   > > > >         +-org.codehaus.jackson:jackson-core-asl:1.9.13
   > > > > ,
   > > > > Require upper bound dependencies error for 
org.codehaus.jackson:jackson-mapper-asl:1.9.2 paths to dependency are:
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-mapper-asl:1.9.2
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-org.apache.avro:avro:1.7.7
   > > > >       +-org.codehaus.jackson:jackson-mapper-asl:1.9.13
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-jaxrs:1.9.13 (managed) <-- 
org.codehaus.jackson:jackson-jaxrs:1.9.2
   > > > >         +-org.codehaus.jackson:jackson-mapper-asl:1.9.13
   > > > > and
   > > > > +-org.apache.uniffle:coordinator:0.8.0-SNAPSHOT
   > > > >   +-org.apache.hadoop:hadoop-common:3.2.1
   > > > >     +-com.sun.jersey:jersey-json:1.19
   > > > >       +-org.codehaus.jackson:jackson-xc:1.9.13 (managed) <-- 
org.codehaus.jackson:jackson-xc:1.9.2
   > > > >         +-org.codehaus.jackson:jackson-mapper-asl:1.9.13
   > > > > ]
   > > > > ```
   > > > 
   > > > 
   > > > It will be ok when you update maven version. You can refer to #685
   > > 
   > > 
   > > But my maven version is just 3.6.3.
   > 
   > Ok, it may be another issue.
   
   
   I think it is another issue indeed. Let's analyse the dependency stack in 
below. 
   In stack1, jackson-mapper-asl is in 4th level, version is 1.9.2. 
   In stack2, jackson-mapper-asl is in 5th level, version is 1.9.13.
   The version break the rule of RequireUpperBoundDeps, the version of 
jackson-mapper-asl in 4th level should be larger than 5th levels's.
   The problem is nothing about uniffle. This is jersey-json's problem, but 
hadoop-3.2 just this jersey-json.
   
   ```
   stack1:
      coordinator        -> 0.8.0-SNAPSHOT
      hadoop-common      -> 3.2.1
      jersey-json        -> 1.19
      jackson-mapper-asl -> 1.9.2
      
   stack2:
      coordinator                               -> 0.8.0-SNAPSHOT
     hadoop-common                      -> 3.2.1
     jersey-json                                -> 1.19
     jackson:jackson-xc         -> 1.9.13
     jackson-mapper-asl         -> 1.9.13
   ```


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