High level C Scan API#7212

Merged

myrrc merged 1 commit intodevelopfrom

Apr 17, 2026

Contributor

myrrc commented Mar 31, 2026 •

edited

Loading

Expose data source, partition, and scan primitives through C with rust-side and C-side tests.

myrrc mentioned this pull request

High level C Scan API vortex-data/rfcs#32

Open

codspeed-hq bot commented Mar 31, 2026 •

edited

Loading

Merging this PR will not alter performance

✅ 1163 untouched benchmarks
⏩ 1457 skipped benchmarks¹

_{Comparing myrrc/scan-ffi (b841add) with develop (cd798b3)}

1457 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

myrrc requested a review from gatesn

March 31, 2026 15:02

myrrc changed the title ~~High level scan api draft~~ High level C Scan API

myrrc force-pushed the myrrc/scan-ffi branch from f741cb7 to ab90339 Compare

April 2, 2026 16:13

myrrc mentioned this pull request

Memory leak while cloning session in FFI runtime #7324

Open

myrrc requested a review from 0ax1

April 8, 2026 14:51

0ax1 reviewed

View reviewed changes

vortex-ffi/cinclude/vortex.h

vortex-ffi/cinclude/vortex.h

vortex-ffi/cinclude/vortex.h Outdated

vortex-ffi/cinclude/vortex.h

vortex-ffi/cinclude/vortex.h

vortex-ffi/cinclude/vortex.h

vortex-ffi/src/scan.rs Outdated

vortex-ffi/src/scan.rs Outdated

vortex-ffi/src/scan.rs Outdated

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/cbindgen.toml

gatesn reviewed

View reviewed changes

vortex-ffi/CMakeLists.txt Outdated

vortex-ffi/src/scan.rs

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/data_source.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

Contributor

0ax1 left a comment

We should prob consider putting #[deny(missing_docs)] for ffi on a module level.

0ax1 reviewed

View reviewed changes

vortex-ffi/test/scan.cpp Outdated

0ax1 reviewed

View reviewed changes

vortex-ffi/src/data_source.rs Outdated

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/src/scan.rs

0ax1 reviewed

View reviewed changes

vortex-ffi/test/scan.cpp Outdated

myrrc force-pushed the myrrc/scan-ffi branch 2 times, most recently from 445319f to ec9a3f3 Compare

April 13, 2026 08:59

myrrc added the changelog/feature label

myrrc force-pushed the myrrc/scan-ffi branch from ec9a3f3 to 5eb62c4 Compare

April 16, 2026 15:19

myrrc marked this pull request as ready for review

April 16, 2026 15:19

myrrc force-pushed the myrrc/scan-ffi branch from 5eb62c4 to 4ffd048 Compare

April 16, 2026 15:19

0ax1 requested review from 0ax1 and gatesn

April 16, 2026 15:20

myrrc force-pushed the myrrc/scan-ffi branch from 4ffd048 to 50dfbd5 Compare

April 16, 2026 15:22

myrrc force-pushed the myrrc/scan-ffi branch from 50dfbd5 to f68d87b Compare

April 16, 2026 15:33

myrrc enabled auto-merge (squash)

April 16, 2026 15:34

myrrc force-pushed the myrrc/scan-ffi branch 2 times, most recently from 26d97d2 to 9483918 Compare

April 17, 2026 09:32


          high level scan api draft

b841add

Signed-off-by: Mikhail Kot <to@myrrc.dev>

myrrc force-pushed the myrrc/scan-ffi branch from 9483918 to b841add Compare

April 17, 2026 09:40

0ax1 approved these changes

View reviewed changes

myrrc merged commit 6a2d4ad into develop

59 of 60 checks passed

myrrc deleted the myrrc/scan-ffi branch

April 17, 2026 09:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature