Skip to content
New issue

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

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

Already on GitHub? # to your account

support to adjust max inflight msgs #450

Merged
merged 7 commits into from
Aug 4, 2021

Conversation

hicqu
Copy link
Contributor

@hicqu hicqu commented Jul 30, 2021

Signed-off-by: qupeng qupeng@pingcap.com

Sometimes we want to custom max-inflight-msgs for a given peer. For example, the target peer's memory or disk usage is almost full. This PR allows this.

BTW a bug in inflight_buffers_size is fixed. With the patch its return value is real size instead of length.

@hicqu hicqu force-pushed the support-adjust-inflight-msgs branch 4 times, most recently from f311a74 to 0fa1d13 Compare August 2, 2021 08:23
Signed-off-by: qupeng <qupeng@pingcap.com>
@hicqu hicqu force-pushed the support-adjust-inflight-msgs branch from 741695c to d9ffd50 Compare August 2, 2021 08:52
Signed-off-by: qupeng <qupeng@pingcap.com>
Signed-off-by: qupeng <qupeng@pingcap.com>
Signed-off-by: qupeng <qupeng@pingcap.com>
BusyJay
BusyJay previously approved these changes Aug 2, 2021
@hicqu
Copy link
Contributor Author

hicqu commented Aug 3, 2021

@jayzhan211 could you take a look again?

Copy link
Contributor

@jayzhan211 jayzhan211 left a comment

Choose a reason for hiding this comment

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

LGTM

@jayzhan211 jayzhan211 self-requested a review August 3, 2021 13:42
jayzhan211
jayzhan211 previously approved these changes Aug 3, 2021
Copy link
Contributor

@jayzhan211 jayzhan211 left a comment

Choose a reason for hiding this comment

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

LGTM

@hicqu hicqu enabled auto-merge (squash) August 4, 2021 03:47
Signed-off-by: qupeng <qupeng@pingcap.com>
@hicqu hicqu dismissed stale reviews from jayzhan211 and BusyJay via 812a317 August 4, 2021 06:19
NingLin-P
NingLin-P previously approved these changes Aug 4, 2021
Copy link
Member

@NingLin-P NingLin-P left a comment

Choose a reason for hiding this comment

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

LGTM

@hicqu hicqu force-pushed the support-adjust-inflight-msgs branch from 6a1c10e to 339abf3 Compare August 4, 2021 06:43
Signed-off-by: qupeng <qupeng@pingcap.com>
@hicqu hicqu force-pushed the support-adjust-inflight-msgs branch from 339abf3 to 91a8237 Compare August 4, 2021 06:54
NingLin-P
NingLin-P previously approved these changes Aug 4, 2021
Signed-off-by: qupeng <qupeng@pingcap.com>
@hicqu hicqu merged commit 15b988a into tikv:master Aug 4, 2021
@hicqu hicqu deleted the support-adjust-inflight-msgs branch August 4, 2021 12:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants