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.

Sign up for GitHub

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

Jump to bottom

Consider re-reorganization the common crates #5019

Closed
BohuTANG opened this issue Apr 24, 2022 · 1 comment · Fixed by #5233
Closed

Consider re-reorganization the common crates #5019

BohuTANG opened this issue Apr 24, 2022 · 1 comment · Fixed by #5233
Assignees
@usamoi
Labels
C-improvement Category: improvement community-take good first issue Category: good first issue

Comments

@BohuTANG
Copy link
Member

Summary

To make common crate more clean and standardized, it would be better to move some root crates in common to common/base:

common/
  - containers
  - range-map
  - infallible
  - mem-allocator

to

common/base/
  - containers
  - range-map
  - infallible
  - mem-allocator
  - base -- this is the what's the common/base/* now
@BohuTANG BohuTANG added C-improvement Category: improvement good first issue Category: good first issue labels Apr 24, 2022
@usamoi
Copy link
Contributor

usamoi commented May 8, 2022

/assignme

@usamoi usamoi mentioned this issue May 8, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@usamoi usamoi

Labels
C-improvement Category: improvement community-take good first issue Category: good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ISSUE-5019: re-reorganization the common crates usamoi/databend
3 participants
@BohuTANG @usamoi @databend-bot