-
Notifications
You must be signed in to change notification settings - Fork 117
[release/0.12] Fix new CI issues #597
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
Conversation
4d0cc57 to
4abf126
Compare
| # Necessary because tokio 1.30.0 updates MSRV to 1.63 | ||
| # and url 2.5.1, updates to 1.67 | ||
| run: | | ||
| cargo update -p parking_lot --precise 0.12.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A lot of crates just recently made the jump from 1.64 to 1.68, parking_lot and tokio moved to 1.71 and even tracing made a very significant change to 1.65.
| // TcpStream::set_linger is deprecated but this use case is valid to reset the stream | ||
| #[allow(deprecated)] | ||
| stream.set_linger(Some(Duration::from_secs(0))).unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that this is more about it being a potential performance issue than anything else, with the thread being potentially blocked on drop.
This specific options will also be now available in new tokio versions as TcpStream::et_linger_zero.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed -- @crepererum fixed the same issue in basically the same way on main in
alamb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @AdamGS
0.12.5(maintenance) - Target Dec 2025 #582This PR fixes some issues that have crept into this branch since the last minor release.