Skip to content

Improve support for sliced input on from_arrow_host APIs#19491

Merged
rapids-bot[bot] merged 29 commits intorapidsai:branch-25.10from
davidwendt:from-arrow-host-sliced
Aug 27, 2025
Merged

Improve support for sliced input on from_arrow_host APIs#19491
rapids-bot[bot] merged 29 commits intorapidsai:branch-25.10from
davidwendt:from-arrow-host-sliced

Conversation

@davidwendt
Copy link
Contributor

Description

Improves support accepting ArrowArray sliced input where the range is beyond the 2 billion row limit.
The length (number of rows) of the sliced input must still be less than 2 billion but the offset/length range may reference indices above 2 billion.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@davidwendt davidwendt self-assigned this Jul 24, 2025
@davidwendt davidwendt added 2 - In Progress Currently a work in progress libcudf Affects libcudf (C++/CUDA) code. improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 24, 2025
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jul 24, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@davidwendt
Copy link
Contributor Author

/ok to test

@davidwendt
Copy link
Contributor Author

/ok to test

@davidwendt
Copy link
Contributor Author

/ok to test

@davidwendt
Copy link
Contributor Author

/ok to test

@davidwendt
Copy link
Contributor Author

/ok to test

Copy link
Contributor

@Matt711 Matt711 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@Matt711 Matt711 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the failing pandas-tests CI job, can you pull in the latest changes from 25.10? So we can see what tests are failing today.

@github-actions github-actions bot added Python Affects Python cuDF API. cudf-polars Issues specific to cudf-polars labels Jul 30, 2025
@GPUtester GPUtester moved this to In Progress in cuDF Python Jul 30, 2025
@davidwendt
Copy link
Contributor Author

/ok to test

@davidwendt davidwendt added the 3 - Ready for Review Ready for review by team label Aug 6, 2025
@davidwendt davidwendt marked this pull request as ready for review August 6, 2025 14:51
@davidwendt davidwendt requested review from a team as code owners August 6, 2025 14:51
Copy link
Contributor

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python changes LGTM

@davidwendt davidwendt requested a review from wence- August 7, 2025 17:01

template <typename T, CUDF_ENABLE_IF(not is_rep_layout_compatible<T>() && !is_fixed_point<T>())>
std::unique_ptr<column> operator()(ArrowSchemaView*, ArrowArray const*, data_type, bool)
std::unique_ptr<column> operator()(ArrowSchemaView const*, ArrowArray const*, data_type, bool)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@davidwendt
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 9fa50b9 into rapidsai:branch-25.10 Aug 27, 2025
93 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in cuDF Python Aug 27, 2025
@davidwendt davidwendt deleted the from-arrow-host-sliced branch August 27, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team cudf-polars Issues specific to cudf-polars improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change Python Affects Python cuDF API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants