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

impl vtab arrow #140

Merged
merged 4 commits into from
Mar 22, 2023
Merged

impl vtab arrow #140

merged 4 commits into from
Mar 22, 2023

Conversation

wangfenjin
Copy link
Collaborator

Change-Id: I2044c36db7d7d145f5def782eb19c45d75ba4747

Change-Id: I2044c36db7d7d145f5def782eb19c45d75ba4747
Change-Id: I3ec9f35a646d23e875807cc7e883e1d3c6b30e80
@codecov
Copy link

codecov bot commented Mar 19, 2023

Codecov Report

Merging #140 (059112c) into main (0c89e7f) will decrease coverage by 1.40%.
The diff coverage is 41.98%.

@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
- Coverage   58.68%   57.29%   -1.40%     
==========================================
  Files          33       34       +1     
  Lines        1847     2028     +181     
==========================================
+ Hits         1084     1162      +78     
- Misses        763      866     +103     
Impacted Files Coverage Δ
libduckdb-sys/src/error.rs 14.28% <ø> (ø)
src/appender.rs 62.50% <ø> (ø)
src/arrow_batch.rs 100.00% <ø> (ø)
src/cache.rs 80.00% <ø> (ø)
src/config.rs 93.33% <ø> (ø)
src/error.rs 20.17% <ø> (-0.55%) ⬇️
src/inner_connection.rs 85.24% <ø> (ø)
src/lib.rs 77.08% <ø> (ø)
src/pragma.rs 68.90% <ø> (ø)
src/r2d2.rs 66.66% <ø> (ø)
... and 16 more

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wangfenjin wangfenjin merged commit f8b45b5 into main Mar 22, 2023
@wangfenjin wangfenjin deleted the vtab-arrow branch March 22, 2023 10:08
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.

1 participant