Skip to content

Lock count guardrail for parallel single phase/sequential task#12052

Merged
jon-wei merged 2 commits intoapache:masterfrom
jon-wei:lock_count_guardrail
Dec 15, 2021
Merged

Lock count guardrail for parallel single phase/sequential task#12052
jon-wei merged 2 commits intoapache:masterfrom
jon-wei:lock_count_guardrail

Conversation

@jon-wei
Copy link
Contributor

@jon-wei jon-wei commented Dec 10, 2021

This PR adds a maxAllowedLockCount tuning config parameter for the parallel single phase and sequential modes that serves as a guardrail on the number of locks that the task set is allowed to acquired. An excessive number of locks can result in a very slow task and excessive load on the overlord.

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@jon-wei jon-wei marked this pull request as ready for review December 10, 2021 19:44
@zachjsh zachjsh self-requested a review December 13, 2021 18:56
Copy link
Contributor

@zachjsh zachjsh left a comment

Choose a reason for hiding this comment

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

LGTM 🚢

@jon-wei jon-wei merged commit 3f79453 into apache:master Dec 15, 2021
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants