streamnative / mop

MQTT on Pulsar implemented using Pulsar Protocol Handler
Apache License 2.0
169 stars 54 forks source link

Add jackson dependency to avoid compile issue. #1406

Closed Technoboy- closed 4 weeks ago

Technoboy- commented 4 weeks ago

Motivation

https://github.com/streamnative/mop/actions/runs/10384444156/job/28751852979?pr=1404

image

The root cause is that, MoP relied on jackson from Pulsar, but jersey-media-json-jackson upgrade to a high version and cause the jackson-databind to 2.16.2. so there throw NoSuchMethodError. EnumResolver.constructUsingToString should be under 2.15

Modifications

Describe the modifications you've done.

Verifying this change

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

Documentation

Check the box below.

Need to update docs?

github-actions[bot] commented 4 weeks ago

@Technoboy-:Thanks for your contribution. For this PR, do we need to update docs? (The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)