Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Added utility to create encodings #1018

Merged
merged 1 commit into from
May 28, 2022
Merged

Added utility to create encodings #1018

merged 1 commit into from
May 28, 2022

Conversation

jorgecarleitao
Copy link
Owner

The new API to write parquet requires a vector of Encoding, one per parquet column.

To help users construct such vector, this PR adds a tool to transverse a field for its (parquet) columns and map the base DataType to the encoding.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label May 28, 2022
@codecov
Copy link

codecov bot commented May 28, 2022

Codecov Report

Merging #1018 (d004720) into main (7cc874f) will increase coverage by 0.03%.
The diff coverage is 21.42%.

@@            Coverage Diff             @@
##             main    #1018      +/-   ##
==========================================
+ Coverage   71.68%   71.71%   +0.03%     
==========================================
  Files         359      359              
  Lines       19880    19949      +69     
==========================================
+ Hits        14251    14307      +56     
- Misses       5629     5642      +13     
Impacted Files Coverage Δ
src/io/parquet/write/mod.rs 57.92% <0.00%> (-3.71%) ⬇️
src/io/parquet/read/deserialize/mod.rs 47.73% <100.00%> (ø)
src/compute/arithmetics/time.rs 26.60% <0.00%> (+0.91%) ⬆️
src/compute/merge_sort/mod.rs 89.10% <0.00%> (+1.74%) ⬆️
src/compute/sort/lex_sort.rs 74.77% <0.00%> (+7.65%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cc874f...d004720. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit f711f84 into main May 28, 2022
@jorgecarleitao jorgecarleitao deleted the added_encoding branch May 28, 2022 04:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant