Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(learn): avoid learn use DEFAULT_POOL #2126

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ruojieranyishen
Copy link
Collaborator

@ruojieranyishen ruojieranyishen commented Sep 25, 2024

FLAGS_max_copy_rate_megabytes_per_disk will use sleep_for function to block THREAD_POOL_DEFAULT. It's cause replica server cannot deal remote command. This modify is same as FLAGS_max_send_rate_megabytes_per_disk, only use THREAD_POOL_BLOCK_SERVICE to sleep.

What problem does this PR solve?

relate issue:#1969

avoid learn use DEFAULT_POOL.

  • Manual test (add detailed scripts or steps below)

Before fix, many partitions ballot increase, and remote_command cannot get response.

0     6       3/3            10.000.000.04:00001  [00.000.00.50:55101,00.000.90.9:55000]     
1     7       3/3            10.000.000.05:00001  [00.000.000.04:55101,10.000.07.9:50001]    
2     5       3/3            10.000.002.07:00001  [00.000.000.04:55101,10.000.00.52:00001]   
3     9       3/3            10.000.000.04:00001  [00.000.00.50:55101,00.000.100.15:00001]

After fix, return to normal.

0     0       3/3            10.000.000.04:00001  [00.000.00.50:55101,00.000.90.9:55000]     
1     0       3/3            10.000.000.05:00001  [00.000.000.04:55101,10.000.07.9:50001]    
2     0       3/3            10.000.002.07:00001  [00.000.000.04:55101,10.000.00.52:00001]   
3     0       3/3            10.000.000.04:00001  [00.000.00.50:55101,00.000.100.15:00001]

@github-actions github-actions bot added the cpp label Sep 25, 2024
lupengfan1 added 2 commits November 28, 2024 10:29
FLAGS_max_copy_rate_megabytes_per_disk will use `sleep_for` function to
block THREAD_POOL_DEFAULT. It's cause replica server cannot deal `remote
command`. This modify is same as FLAGS_max_send_rate_megabytes_per_disk,
only use THREAD_POOL_BLOCK_SERVICE to sleep.

relate issue:apache#1969
@ruojieranyishen ruojieranyishen force-pushed the avoid_learn_use_default_pool branch from 37b4579 to 5f338df Compare November 28, 2024 02:29
@ruojieranyishen ruojieranyishen changed the title fix(learn) avoid learn use DEFAULT_POOL fix(learn): avoid learn use DEFAULT_POOL Nov 28, 2024
acelyc111
acelyc111 previously approved these changes Dec 4, 2024
@acelyc111
Copy link
Member

@ruojieranyishen Thanks for the contribution!

Please fix the CI errors.

src/nfs/nfs_code_definition.h Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants