anupamaggarwal commented on code in PR #25890:
URL: https://github.com/apache/flink/pull/25890#discussion_r1912830806


##########
pom.xml:
##########
@@ -328,6 +335,13 @@ under the License.
                -->
                <dependencies>
 
+                       <!-- Setting default version for jackson-mapper-asl -->
+                       <dependency>
+                               <groupId>org.codehaus.jackson</groupId>
+                               <artifactId>jackson-mapper-asl</artifactId>
+                               <version>${jackson.mapper.asl.version}</version>
+                       </dependency>
+

Review Comment:
   thanks @rmetzger 
   This should not be the case since it's included only in the 
`dependencyManagement` section.
   My understanding is that in this mode the dependency will only be introduced 
if a child pom directly refers to `jackson-mapper-asl`, and not otherwise (Also 
the child pom does not have to specify the version of jackson-mapper-asl and it 
will be determined from the parent pom) 



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

Reply via email to