Skip to content

KAFKA-12856: Upgrade Jackson to 2.12.3#10778

Merged
ijuma merged 1 commit into
apache:trunkfrom
ijuma:jackson-upgrade
May 27, 2021
Merged

KAFKA-12856: Upgrade Jackson to 2.12.3#10778
ijuma merged 1 commit into
apache:trunkfrom
ijuma:jackson-upgrade

Conversation

@ijuma

@ijuma ijuma commented May 27, 2021

Copy link
Copy Markdown
Member

2.10.x is no longer supported, so we should move to 2.12 for the 3.0
release.

ScalaObjectMapper has been deprecated and it looks like we don't
actually need it, so remove its usage.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

2.10.x is no longer supported, so we should move to 2.12 for the 3.0
release.

ScalaObjectMapper has been deprecated and it looks like we don't
actually need it, so remove its usage.

@dajac dajac left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, assuming tests pass.

@ijuma

ijuma commented May 27, 2021

Copy link
Copy Markdown
Member Author

Tests look good, one job passed, the others had unrelated failures:

Build / JDK 15 and Scala 2.13 / testCreateClusterAndCreateAndManyTopicsWithManyPartitions() – kafka.server.RaftClusterTest
23s
Build / JDK 15 and Scala 2.13 / testCreateClusterAndCreateAndManyTopicsWithManyPartitions() – kafka.server.RaftClusterTest
18s
Build / JDK 8 and Scala 2.12 / testCreateClusterAndCreateAndManyTopicsWithManyPartitions() – kafka.server.RaftClusterTest
25s
Build / JDK 8 and Scala 2.12 / testCreateClusterAndCreateAndManyTopicsWithManyPartitions() – kafka.server.RaftClusterTest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants