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
Reduce trivial async
block to reduce future size and save memory usage.
#16540
Closed
Tracked by
#16375
Labels
Comments
15 tasks
2 tasks
ti-chi-bot bot
added a commit
that referenced
this issue
Feb 21, 2024
close #16540 *: enable linters about async and futures We should be pedantic about writing async code, as it's easy to write suboptimal or even bloat code. See: rust-lang/rust#69826 *: remove unnecessary async blocks to save memory This commit favors FutureExt::map over async blocks to mitigate the issue of async block doubled memory usage. Through the sysbench oltp_read_only test, it was observed that this adjustment resulted in approximately 26% reduction in memory usage. See: rust-lang/rust#59087 Signed-off-by: Neil Shen <overvenus@gmail.com> Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 13, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot
pushed a commit
to ti-chi-bot/tikv
that referenced
this issue
Mar 18, 2024
close tikv#16540 Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
2 tasks
ti-chi-bot bot
pushed a commit
that referenced
this issue
Mar 18, 2024
close #16540 *: enable linters about async and futures We should be pedantic about writing async code, as it's easy to write suboptimal or even bloat code. See: rust-lang/rust#69826 *: remove unnecessary async blocks to save memory This commit favors FutureExt::map over async blocks to mitigate the issue of async block doubled memory usage. Through the sysbench oltp_read_only test, it was observed that this adjustment resulted in approximately 26% reduction in memory usage. See: rust-lang/rust#59087 Signed-off-by: Neil Shen <overvenus@gmail.com> Co-authored-by: Neil Shen <overvenus@gmail.com>
dbsid
pushed a commit
to dbsid/tikv
that referenced
this issue
Mar 24, 2024
close tikv#16540 *: enable linters about async and futures We should be pedantic about writing async code, as it's easy to write suboptimal or even bloat code. See: rust-lang/rust#69826 *: remove unnecessary async blocks to save memory This commit favors FutureExt::map over async blocks to mitigate the issue of async block doubled memory usage. Through the sysbench oltp_read_only test, it was observed that this adjustment resulted in approximately 26% reduction in memory usage. See: rust-lang/rust#59087 Signed-off-by: Neil Shen <overvenus@gmail.com> Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com> Signed-off-by: dbsid <chenhuansheng@pingcap.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
See more: rust#59087
The text was updated successfully, but these errors were encountered: