-
Notifications
You must be signed in to change notification settings - Fork 333
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
feat: convert FileMetaData
to ParquetMetaData
#2980
feat: convert FileMetaData
to ParquetMetaData
#2980
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2980 +/- ##
===========================================
- Coverage 85.85% 85.24% -0.62%
===========================================
Files 775 784 +9
Lines 124707 126760 +2053
===========================================
+ Hits 107070 108053 +983
- Misses 17637 18707 +1070 |
Co-authored-by: fys <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM.
Co-authored-by: Yingwen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: convert `FileMetaData` to `ParquetMetaData` (GreptimeTeam#2980) * feat: can convert Format FileMetaData to ParquetMetaData * test: parquet metadata equal * chore: test * chore: cr comment Co-authored-by: fys <[email protected]> * chore: cr comment * refactor: type name * chore: cr comment Co-authored-by: Yingwen <[email protected]> --------- Co-authored-by: fys <[email protected]> Co-authored-by: Yingwen <[email protected]> * refactor(remote_wal): add StandaloneWalConfig (GreptimeTeam#3002) * feat: integrate remote wal to standalone * fix: test * refactor: refactor standalone wal config * chore: change default kafka port to 9092 * chore: apply suggestions from CR --------- Co-authored-by: niebayes <[email protected]> --------- Co-authored-by: Wei <[email protected]> Co-authored-by: fys <[email protected]> Co-authored-by: Yingwen <[email protected]> Co-authored-by: Weny Xu <[email protected]> Co-authored-by: niebayes <[email protected]>
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
part of #2963 , this pr is pre-work.
support to convert from
FileMetaData
toParquetMetaData
.Checklist
Refer to a related PR or issue link (optional)
#2963