search

risingwavelabs / risingwave

Best-in-class stream processing, analytics, and management. Perform continuous analytics, or build event-driven applications, real-time ETL pipelines, and feature stores in minutes. Unified streaming and batch. PostgreSQL compatible.
https://www.risingwave.com/slack
Apache License 2.0
6.75k stars 557 forks source link

style: refactor async trait with `async-fn-in-trait` and `return-position-impl-trait-in-trait` #10554

Open BugenZhao opened 1 year ago

BugenZhao commented 1 year ago

These features are still unstable but not considered to be "incomplete" now! We can use them to simplify the async trait interfaces.

Already done:

By searching the codebase with type .+ = impl .*Future, I find that the storage module is the most significant and valuable part to refactor. cc @wenym1

github-actions[bot] commented 2 months ago

This issue has been open for 60 days with no activity.

If you think it is still relevant today, and needs to be done in the near future, you can comment to update the status, or just manually remove the no-issue-activity label.

You can also confidently close this issue as not planned to keep our backlog clean. Don't worry if you think the issue is still valuable to continue in the future. It's searchable and can be reopened when it's time. 😄