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

Made multiversion and lexical-core optional #324

Merged
merged 1 commit into from
Aug 24, 2021
Merged

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Aug 23, 2021

Made multiversion and lexical-core optional and dependent on whether they are needed.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 23, 2021
@jorgecarleitao jorgecarleitao changed the title Reduced dependencies. Made mutiversion and lexical-core optional Aug 23, 2021
@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #324 (fb716e3) into main (824ad7e) will increase coverage by 0.05%.
The diff coverage is 62.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #324      +/-   ##
==========================================
+ Coverage   80.55%   80.61%   +0.05%     
==========================================
  Files         324      324              
  Lines       21399    21562     +163     
==========================================
+ Hits        17239    17383     +144     
- Misses       4160     4179      +19     
Impacted Files Coverage Δ
src/util/lexical.rs 62.50% <62.50%> (ø)
src/io/parquet/read/boolean/basic.rs 84.61% <0.00%> (-3.53%) ⬇️
tests/it/io/ipc/common.rs 100.00% <0.00%> (ø)
tests/it/io/ipc/write/stream.rs 100.00% <0.00%> (ø)
src/io/parquet/read/fixed_size_binary.rs 43.92% <0.00%> (+0.92%) ⬆️
src/io/parquet/read/binary/basic.rs 82.75% <0.00%> (+1.42%) ⬆️
src/io/ipc/read/stream.rs 83.76% <0.00%> (+1.48%) ⬆️
src/io/parquet/read/primitive/basic.rs 82.92% <0.00%> (+1.53%) ⬆️
src/io/parquet/read/primitive/dictionary.rs 77.50% <0.00%> (+7.35%) ⬆️
... and 1 more

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 824ad7e...fb716e3. Read the comment docs.

@jorgecarleitao jorgecarleitao changed the title Made mutiversion and lexical-core optional Made multiversion and lexical-core optional Aug 24, 2021
@jorgecarleitao jorgecarleitao changed the title Made multiversion and lexical-core optional Made multiversion and lexical-core optional Aug 24, 2021
@jorgecarleitao jorgecarleitao merged commit 77a2934 into main Aug 24, 2021
@jorgecarleitao jorgecarleitao deleted the dependencies branch August 24, 2021 07:50
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