bbende commented on a change in pull request #5630:
URL: https://github.com/apache/nifi/pull/5630#discussion_r778858154



##########
File path: nifi-manifest/nifi-extension-manifest-parser/pom.xml
##########
@@ -29,11 +29,23 @@
             <artifactId>nifi-extension-manifest-model</artifactId>
             <version>1.16.0-SNAPSHOT</version>
         </dependency>
-        <dependency>
-            <groupId>com.fasterxml.jackson.dataformat</groupId>
-            <artifactId>jackson-dataformat-xml</artifactId>

Review comment:
       Yes, this module needs to be used from the framework and 
jackson-dataformat-xml brings in transitive dependency for woodstox which then 
gets found as an XML provider for other things that it wasn't intended to be 
found for. Using JAX-B seemed like a simpler approach since everything is 
already setup for it being available on the classpath.




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