Skip to content

KAFKA-2613: Make maxParallelForks configurable via Gradle config so it can be turned down on shared build infrastructure.#293

Closed
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-2613-user-configurable-max-forks
Closed

KAFKA-2613: Make maxParallelForks configurable via Gradle config so it can be turned down on shared build infrastructure.#293
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-2613-user-configurable-max-forks

Conversation

@ewencp
Copy link
Copy Markdown
Contributor

@ewencp ewencp commented Oct 9, 2015

No description provided.

…t can be turned down on shared build infrastructure.
@ijuma
Copy link
Copy Markdown
Member

ijuma commented Oct 10, 2015

The change looks good. Did you test that the number of processes corresponds to the value passed via the property?

@ewencp
Copy link
Copy Markdown
Contributor Author

ewencp commented Oct 10, 2015

Yes, just did a manual check with ps and a bit of grep and wc. Was at 8 when I ran without the flag, dropped to the target number with the flag.

@ijuma
Copy link
Copy Markdown
Member

ijuma commented Oct 10, 2015

Sounds good.

@harshach
Copy link
Copy Markdown

+1

@asfgit asfgit closed this in d734050 Oct 13, 2015
@gwenshap
Copy link
Copy Markdown
Contributor

Merged. @harshach, just a reminder that you have a commit bit - so if you can +1 something you can also merge :)

wyuka pushed a commit to wyuka/kafka that referenced this pull request Mar 4, 2022
…ache#293)

TICKET = N/A
LI_DESCRIPTION = vanilla logic can lead to clients being sticky to the same broker, which can cause issues. added new mode to spread out the MD requests a bit more.

fix merge conflicts
EXIT_CRITERIA = N/A

(cherry picked from commit 0d2abca)

Co-authored-by: Radai Rosenblatt <radai.rosenblatt@gmail.com>
wyuka pushed a commit to wyuka/kafka that referenced this pull request Mar 28, 2022
…ache#293)

TICKET = N/A
LI_DESCRIPTION = vanilla logic can lead to clients being sticky to the same broker, which can cause issues. added new mode to spread out the MD requests a bit more.

fix merge conflicts
EXIT_CRITERIA = N/A

(cherry picked from commit 0d2abca)

Co-authored-by: Radai Rosenblatt <radai.rosenblatt@gmail.com>
wyuka pushed a commit to wyuka/kafka that referenced this pull request Jun 16, 2022
…ache#293)

TICKET = N/A
LI_DESCRIPTION = vanilla logic can lead to clients being sticky to the same broker, which can cause issues. added new mode to spread out the MD requests a bit more.

fix merge conflicts
EXIT_CRITERIA = N/A

(cherry picked from commit 0d2abca)

Co-authored-by: Radai Rosenblatt <radai.rosenblatt@gmail.com>
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
…ts (apache#293)

* Test that unrelated batches are unaffected in merging

* Small code improvements in commit_file_merge_work_item_v1
jeqo pushed a commit to aiven/kafka that referenced this pull request Jan 16, 2026
…ts (apache#293)

* Test that unrelated batches are unaffected in merging

* Small code improvements in commit_file_merge_work_item_v1
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.

4 participants