-
Notifications
You must be signed in to change notification settings - Fork 5.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
hide & deprecate enable-streaming #20759
Comments
/cc @zz-jason @breeswish |
@SunRunAway could you link all the bugs that related to streaming API? Without seeing this bug list, it's hard to decide whether to deprecate it temporarily. |
We are currently focusing on optimizing the non-streaming coprocessor API for its performance and resource usage, but when the users enable the steaming API, they will not be benefited from us. |
@zz-jason There are a lot of known trivial issues, for example,
In summary, the streaming API seems to be not complete and not efficient. There is no plan to optimize. Tests may not completely covered as well. |
@breeswish got it, thanks. |
Feature Request
Is your feature request related to a problem? Please describe:
enable-streaming
is not well maintained because we are focusing on optimizing the non-streaming coprocessor API for its performance and resource usage and don’t have time to optimize streaming's currently, I propose to deprecate it temporarily until somebody has time to pick it up again and prove it could be GA in the future.Describe the feature you'd like:
Describe alternatives you've considered:
Teachability, Documentation, Adoption, Migration Strategy:
see https://asktug.com/t/topic/37526/7
The text was updated successfully, but these errors were encountered: