Skip to content
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

refactor: Context component on read / write path #214

Open
ethe opened this issue Nov 12, 2024 · 0 comments
Open

refactor: Context component on read / write path #214

ethe opened this issue Nov 12, 2024 · 0 comments
Labels
M - Medium Adding a new feature, refactoring a module, performance optimization. refactor No more features, just refactoring

Comments

@ethe
Copy link
Member

ethe commented Nov 12, 2024

Tonbo's read and write path needs a lot of context (version, storage manager and lru cache), such as storage manager, it might be used in the bottom of the read path but be initialized in the top-level (db instances). In the current, we just simply pass through them from top-level to bottom-level, it is ugly and expose implementations to those context agnostic layers. Therefore, use a unified context component to manage those would be better.

@ethe ethe added M - Medium Adding a new feature, refactoring a module, performance optimization. refactor No more features, just refactoring labels Nov 12, 2024
@ethe ethe changed the title Feat: Context component on read / write path Refactor: Context component on read / write path Nov 12, 2024
@ethe ethe added this to Tonbo Nov 14, 2024
@ethe ethe moved this to Todo in Tonbo Nov 14, 2024
@ethe ethe changed the title Refactor: Context component on read / write path refactor: Context component on read / write path Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M - Medium Adding a new feature, refactoring a module, performance optimization. refactor No more features, just refactoring
Projects
Status: Todo
Development

No branches or pull requests

1 participant