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: bump arrow, parquet, datafusion and tonic #1386

Merged
merged 4 commits into from
Apr 14, 2023

Conversation

waynexia
Copy link
Member

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

PLEASE DO NOT LEAVE THIS EMPTY !!!

  • Update
    • arrow to 37
    • parquet to 37
    • datafusion to 74a778ca6016a853a3c3add3fa8c6f12f4fe4561
    • tonic to 0.9
    • greptime-proto to 0bebe5f69c91cdfbce85cb8f45f9fcd28185261c
  • Fix

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)

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Copy link
Contributor

@killme2008 killme2008 left a comment

Choose a reason for hiding this comment

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

Looks very good to me!

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
@codecov
Copy link

codecov bot commented Apr 14, 2023

Codecov Report

Merging #1386 (8345efc) into develop (43391e0) will decrease coverage by 0.54%.
The diff coverage is 84.87%.

@@             Coverage Diff             @@
##           develop    #1386      +/-   ##
===========================================
- Coverage    86.06%   85.52%   -0.54%     
===========================================
  Files          529      530       +1     
  Lines        78770    78922     +152     
===========================================
- Hits         67792    67499     -293     
- Misses       10978    11423     +445     

@killme2008 killme2008 requested review from v0y4g3r and evenyag April 14, 2023 12:48
Copy link
Contributor

@v0y4g3r v0y4g3r left a comment

Choose a reason for hiding this comment

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

LGTM

@v0y4g3r v0y4g3r merged commit a6e41cd into GreptimeTeam:develop Apr 14, 2023
@waynexia waynexia deleted the bump-deps branch April 17, 2023 02:35
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
* bump arrow, parquet, datafusion, tonic and greptime-proto

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* add analyzer and fix test

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* fix clippy warnings

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* update sqlness result

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

---------

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
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