Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ti-chi-bot
Copy link
Member

This is an automated cherry-pick of #4074

What problem does this PR solve?

when initializing the encoder, make sure max-message-bytes in the opts, and set identical to producer's configuration.

What is changed and how it works?

Check List

Tests

  • Unit test

Code changes

Side effects

Related changes

Release note

`None`.

3AceShowHand and others added 30 commits November 22, 2021 05:33
Copy link
Contributor

@amyangfei amyangfei left a comment

Choose a reason for hiding this comment

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

rest lgtm

@ti-chi-bot ti-chi-bot added status/LGT2 Indicates that a PR has LGTM 2. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed status/LGT1 Indicates that a PR has LGTM 1. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Dec 27, 2021
@3AceShowHand 3AceShowHand force-pushed the cherry-pick-4074-to-release-5.1 branch from 9c505b1 to 4bf83dc Compare December 27, 2021 05:03
@ti-chi-bot ti-chi-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Dec 27, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2021

Codecov Report

❗ No coverage uploaded for pull request base (release-5.1@1ac656b). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-5.1      #4077   +/-   ##
================================================
  Coverage               ?   64.1878%           
================================================
  Files                  ?        160           
  Lines                  ?      16782           
  Branches               ?          0           
================================================
  Hits                   ?      10772           
  Misses                 ?       5071           
  Partials               ?        939           

@3AceShowHand
Copy link
Contributor

/run-integration-test
/run-kafka-integration-test

@0xPoe
Copy link
Member

0xPoe commented Dec 27, 2021

/merge

@ti-chi-bot
Copy link
Member Author

This pull request has been accepted and is ready to merge.

Commit hash: bd8a6e1

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Dec 27, 2021
@zhouqiang-cl zhouqiang-cl added the cherry-pick-approved Cherry pick PR approved by release team. label Dec 27, 2021
@0xPoe
Copy link
Member

0xPoe commented Dec 27, 2021

/hold

Waiting for manual testing

@ti-chi-bot ti-chi-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 27, 2021
@3AceShowHand
Copy link
Contributor

/run-all-tests

@3AceShowHand
Copy link
Contributor

/run-kafka-integration-test

@overvenus
Copy link
Member

/unhold

@ti-chi-bot ti-chi-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 27, 2021
@overvenus
Copy link
Member

/merge

@ti-chi-bot ti-chi-bot merged commit d94cf7c into pingcap:release-5.1 Dec 27, 2021
@overvenus overvenus added this to the v5.1.4 milestone Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick-approved Cherry pick PR approved by release team. release-note-none Denotes a PR that doesn't merit a release note. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug. type/cherry-pick-for-release-5.1 This PR is cherry-picked to release-5.1 from a source PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants