rdblue commented on pull request #2479: URL: https://github.com/apache/iceberg/pull/2479#issuecomment-821909340
Okay, if I'm reading this thread correctly, the relevant information is: * Parquet moved to the newer Jackson package names, in `com.fasterxml.jackson` * Parquet encryption has an undeclared transitive dependency on classes in the old `org.codehaus.jackson` modues * Our re-shading of `org.codehaus.jackson` breaks Parquet encryption because Parquet no longer produces `parquet.org.codehaus.jackson` and we assume that it does Is Avro also broken? If not, why not? We still need to avoid leaking `org.codehaus.jackson` from Avro. It looks like we now need to include a copy and relocate it. That should fix both Avro and the Parquet classes that depend on the older Jackson. @flyrain, how long has this been a problem? Do we need to fix master or are the 0.11.x releases also affected? -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
