You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[C++] Switch bindings-scanner to the new scan node, remove the old scan node #32987
Once the new scan node is fully capable we should switch the language bindings (pyarrow / R) to use the new scan options. At this point we can get rid of the old scan node.
We will probably need to go through a deprecation cycle to remove the old scan node since it has a different API. I'm going to start listing the things we want to deprecate here:
We can deprecate the old scanning methods in fragment / dataset (e.g. FileFormat::Inspect or Fragment::ScanBatchesAsync.
We can deprecate the GetRecordBatchGenerator method in parquet::arrow::FileReader (in favor of ReadRowGroupsAsync)
This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.
Activity
westonpace commentedon Jul 18, 2023
We will probably need to go through a deprecation cycle to remove the old scan node since it has a different API. I'm going to start listing the things we want to deprecate here:
FileFormat::InspectorFragment::ScanBatchesAsync.GetRecordBatchGeneratormethod inparquet::arrow::FileReader(in favor ofReadRowGroupsAsync)thisisnic commentedon Sep 23, 2026
This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.