search

datafusion-contrib / datafusion-orc

Implementation of Apache ORC file format use Apache Arrow in-memory format
Apache License 2.0
41 stars 10 forks source link

feat: support to map datatype #32

Closed WenyXu closed 10 months ago

WenyXu commented 10 months ago

related #14

codecov[bot] commented 10 months ago

Codecov Report

Merging #32 (424b021) into main (b78e9da) will increase coverage by 0.77%. Report is 1 commits behind head on main. The diff coverage is 79.13%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #32 +/- ## ========================================== + Coverage 76.44% 77.22% +0.77% ========================================== Files 32 34 +2 Lines 3210 3302 +92 ========================================== + Hits 2454 2550 +96 + Misses 756 752 -4 ```
Jefffrey commented 10 months ago

I will take a look later today (or over weekend) :+1:

Jefffrey commented 10 months ago

I'll hold off on reviewing this further. Need this updated with main and also after https://github.com/datafusion-contrib/datafusion-orc/pull/30 is merged, since it seems to duplicate some changes

WenyXu commented 10 months ago

@Jefffrey PTAL