-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
str #19589 backport autoflushing in outputstream sink #20164
Conversation
But this changes the signature... |
Nope, it's safe, specifically adds overload with the additional boolean (instead of changing signature to use default param, which would be breaking), internal classes constructor param we don't guarantee anything about (though you're right MiMa will panic, I'll fix that). |
Test PASSed. |
Test FAILed. |
Actual perhaps failure?
|
I believe this failure has nothing to do with the backport. @ktoso can you please rebase it? If after rebase validation passes, I opt for merging this. |
rebased |
Test FAILed. |
Failure was unrelated #20197 |
Manual backport of the auto flushing feature added in #19589
Just cherry picking is not as trivial since we introduced more things (IOResult etc).