Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CC-22326 | Bump up jackson version to increase max string value length from 5 to 20 million chars #716

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

pbadani
Copy link
Member

@pbadani pbadani commented Sep 8, 2023

Problem

Connector fails with payloads > 5MB

Solution

Bump up jackson dependency to include the following fix FasterXML/jackson-core@0484754

Does this solution apply anywhere else?
  • yes
  • no
If yes, where?

Test Strategy

Testing done:
  • Unit tests
  • Integration tests
  • System tests
  • Manual tests - Docker Playground.

Release Plan

@pbadani pbadani requested a review from a team as a code owner September 8, 2023 11:39
@cla-assistant
Copy link

cla-assistant bot commented Sep 8, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@cla-assistant
Copy link

cla-assistant bot commented Sep 8, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@pbadani pbadani requested a review from sp-gupta September 8, 2023 11:40
@pbadani pbadani merged commit 149d338 into 12.0.x Sep 12, 2023
@pbadani pbadani deleted the pbadani/CC-22326 branch September 12, 2023 07:06
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