Skip to content

When the traffic to be unloaded is too small, it means that maxThroug… - #13140

Closed
lordcheng10 wants to merge 3 commits into
apache:masterfrom
lordcheng10:add_minThroughputThreshold_in_uniformloadshedder
Closed

When the traffic to be unloaded is too small, it means that maxThroug…#13140
lordcheng10 wants to merge 3 commits into
apache:masterfrom
lordcheng10:add_minThroughputThreshold_in_uniformloadshedder

Conversation

@lordcheng10

@lordcheng10 lordcheng10 commented Dec 5, 2021

Copy link
Copy Markdown
Contributor

Motivation

When the traffic to be unloaded is too small, it means that maxThroughputRate is very small, and there is no need to unload at this time to avoid unnecessary unload.

Need to update docs?

  • no-need-doc

…hputRate is very small, and there is no need to unload at this time to avoid unnecessary unload
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Dec 5, 2021
@lordcheng10

lordcheng10 commented Dec 5, 2021

Copy link
Copy Markdown
Contributor Author

@rdhabalia This is an optimization of the #12902
PTAL,thanks!

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

11 similar comments
@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@lordcheng10

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@rdhabalia

Copy link
Copy Markdown
Contributor

let's first merge #13080 to handle the basic scenario of the strategy before bringing any enhancement.

@github-actions

Copy link
Copy Markdown

The pr had no activity for 30 days, mark with Stale label.

@github-actions

Copy link
Copy Markdown

The pr had no activity for 30 days, mark with Stale label.

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

Labels

doc-not-needed Your PR changes do not impact docs lifecycle/stale Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants