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

Added support for random access reads from IPC #1034

Merged
merged 1 commit into from
May 31, 2022
Merged

Conversation

jorgecarleitao
Copy link
Owner

This exposes an API to read from IPC on a random-access fashion, i.e. read record batch at position i.

@jorgecarleitao jorgecarleitao added the feature A new feature label May 31, 2022
@jorgecarleitao jorgecarleitao changed the title Added ability for random access reads to IPC Added support for random access reads to IPC May 31, 2022
@codecov
Copy link

codecov bot commented May 31, 2022

Codecov Report

Merging #1034 (ec383ad) into main (62f5892) will increase coverage by 0.26%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main    #1034      +/-   ##
==========================================
+ Coverage   72.36%   72.63%   +0.26%     
==========================================
  Files         359      359              
  Lines       19926    19917       -9     
==========================================
+ Hits        14419    14466      +47     
+ Misses       5507     5451      -56     
Impacted Files Coverage Δ
src/io/ipc/read/reader.rs 78.43% <70.58%> (+2.63%) ⬆️
src/io/ipc/append/mod.rs 60.00% <100.00%> (ø)
src/ffi/schema.rs 90.96% <0.00%> (+15.80%) ⬆️

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 62f5892...ec383ad. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 47d5bd2 into main May 31, 2022
@jorgecarleitao jorgecarleitao deleted the ipc_random branch May 31, 2022 17:22
@jorgecarleitao jorgecarleitao changed the title Added support for random access reads to IPC Added support for random access reads from IPC Jun 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant