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

chore: reduce the number of requests for meta #1647

Merged

Conversation

fengys1996
Copy link
Contributor

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

This pr mainly reduce the number of requests for meta.

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@fengys1996 fengys1996 marked this pull request as ready for review May 26, 2023 02:23
Copy link
Contributor

@shuiyisong shuiyisong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fengys1996 fengys1996 marked this pull request as draft May 26, 2023 02:44
@fengys1996 fengys1996 force-pushed the feat/reduce/meta_access branch from b06d148 to 0e7e134 Compare May 26, 2023 09:03
@fengys1996 fengys1996 marked this pull request as ready for review May 26, 2023 09:04
@codecov
Copy link

codecov bot commented May 26, 2023

Codecov Report

Merging #1647 (0e7e134) into develop (8f9e968) will decrease coverage by 0.58%.
The diff coverage is 96.87%.

@@             Coverage Diff             @@
##           develop    #1647      +/-   ##
===========================================
- Coverage    85.80%   85.22%   -0.58%     
===========================================
  Files          563      566       +3     
  Lines        90133    90386     +253     
===========================================
- Hits         77335    77032     -303     
- Misses       12798    13354     +556     

@MichaelScofield MichaelScofield merged commit f0a519b into GreptimeTeam:develop May 26, 2023
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants