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

Improvement: Remove dependency parquet-format-async-temp #4675

Closed
dantengsky opened this issue Apr 2, 2022 · 1 comment
Closed

Improvement: Remove dependency parquet-format-async-temp #4675

dantengsky opened this issue Apr 2, 2022 · 1 comment
Assignees
@dantengsky
Labels
A-storage Area: databend storage C-feature Category: feature C-improvement Category: improvement

Comments

@dantengsky
Copy link
Member

Summary

currently parquet-format-async-temp (0.2.0) is explicitly dependent.

which introduces troubles in the upgrading of crate parquet2 (which depends on parquet-format-async-temp 0.3.0 now).

and also, It seems that we should use the interfaces exported by arrow2 to manipulate parquet.
@dantengsky dantengsky added C-feature Category: feature A-storage Area: databend storage C-improvement Category: improvement labels Apr 2, 2022
@dantengsky dantengsky self-assigned this Apr 2, 2022
@dantengsky dantengsky mentioned this issue Apr 6, 2022
Merged
3 tasks
@Xuanwo
Copy link
Member

Xuanwo commented Apr 8, 2022

Fixed in #4726, closing now.

@Xuanwo Xuanwo closed this as completed Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dantengsky dantengsky

Labels
A-storage Area: databend storage C-feature Category: feature C-improvement Category: improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Xuanwo @dantengsky