gnodet commented on PR #21800: URL: https://github.com/apache/camel/pull/21800#issuecomment-4038686971
## Final Analysis: Ready to Merge ### Jackson 2 BOM The `com.fasterxml.jackson:jackson-bom` import in `parent/pom.xml` **should be kept**. Several modules still depend on Jackson 2, and the BOM ensures version alignment across all `com.fasterxml.jackson.*` artifacts — including transitive dependencies pulled in by Swagger, OpenSearch, JSLT, and Quarkus. Managing those individually would be fragile and error-prone. This is temporary until upstream libraries release Jackson 3-compatible versions. ### Remaining modules (blocked by upstream) | Module | Blocker | |--------|---------| | **camel-jackson** | Intentional — Jackson 2 data format (parallel `camel-jackson3` exists) | | **camel-jacksonxml** | Intentional — Jackson 2 XML data format (parallel `camel-jackson3xml` exists) | | **camel-jslt** | JSLT library (0.1.14) depends on `com.fasterxml.jackson.core:jackson-databind` at runtime | | **camel-opensearch** | OpenSearch client (3.7.0) depends on Jackson 2 | | **camel-openapi-java** | Swagger libraries (`swagger-core-jakarta` 2.2.36) depend on Jackson 2 | | **camel-jbang-mcp** | No direct source usage — Swagger + Quarkus transitive deps require Jackson 2 (test-scope only) | ### Recommendation This PR should be **merged as-is**. The Jackson 3 migration is as complete as it can be — all remaining Jackson 2 usages are either intentional (dual Jackson 2/3 data format components) or blocked by upstream libraries that haven't released Jackson 3-compatible versions yet. Those can be migrated incrementally as upstream catches up. -- 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]
