-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[REMOVE] Cleanup deprecated thread pool settings #2595
Comments
It looks like the AutoQueueAdjustingExecutorBuilder itself was deprecated for removal due to being experimental and not being used. Seems like we can go ahead and take it out unless anyone has objections. |
I don't think we need to get this change into the upcoming 2.0 release, but we can target it for 2.1. |
It will be breaking change, right? So next major like 3.0.0? |
Of course, you're right, since there are configuration settings to remove. I still think we can pass on 2.0 since that is coming up rapidly and this doesn't seem critical. We can target 3.0 instead. |
Is your feature request related to a problem? Please describe.
Right now some of the thread pool settings are declared as deprecated and logged by deprecation logger:
Describe the solution you'd like
Cleanup deprecated thread pool settings in 3.0.0
Describe alternatives you've considered
Leave them there
Additional context
N/A
The text was updated successfully, but these errors were encountered: