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 replacing metasrv flight API with bare grpc? #1927

Closed
drmingdrmer opened this issue Sep 22, 2021 · 3 comments
Closed

Consider replacing metasrv flight API with bare grpc? #1927

drmingdrmer opened this issue Sep 22, 2021 · 3 comments
Labels
A-meta Area: databend meta serive C-improvement Category: improvement
Projects
Meta Service Store Refactor

Comments

@drmingdrmer
Copy link
Member

To access meta data(purely key-value API), it does not need to follow the arrow-flight protocol, which is designed for both meta data and stream data transfer.
@drmingdrmer drmingdrmer added C-improvement Category: improvement A-meta Area: databend meta serive labels Sep 22, 2021
@BohuTANG
Copy link
Member

BohuTANG commented Sep 22, 2021 •
edited

arrow flight is for data transfer.
For metasvr, grpc + timeout+sync-style is ok(a big change).

@BohuTANG BohuTANG added this to To do in Store Refactor Sep 23, 2021
@drmingdrmer drmingdrmer moved this from To do to Planning in Store Refactor Sep 23, 2021
@dantengsky
Copy link
Member

make sense

@dantengsky dantengsky mentioned this issue Sep 24, 2021
Merged
@ariesdevil
Copy link
Member

closed by #3606

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-meta Area: databend meta serive C-improvement Category: improvement
Projects
No open projects
Meta Service
Awaiting triage
Store Refactor
Planning
Milestone
No milestone
Development

No branches or pull requests
4 participants
@drmingdrmer @BohuTANG @ariesdevil @dantengsky