Skip to content

Make copy-on-write the default for pandas3 - #21801

Merged
vyasr merged 2 commits into
NVIDIA:pandas3from
vyasr:feat/cow_default
Mar 16, 2026
Merged

Make copy-on-write the default for pandas3#21801
vyasr merged 2 commits into
NVIDIA:pandas3from
vyasr:feat/cow_default

Conversation

@vyasr

@vyasr vyasr commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

Description

This PR removes all configurability around copy-on-write and turns it on unconditionally. I didn't remove some of the behavioral tests for CoW so we still have some level of testing of CoW correctness.

Checklist

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

@vyasr vyasr self-assigned this Mar 16, 2026
@vyasr
vyasr requested a review from a team as a code owner March 16, 2026 22:49
@vyasr
vyasr requested review from bdice and mroeschke and removed request for a team March 16, 2026 22:49
@vyasr vyasr added improvement Improvement / enhancement to an existing function breaking Breaking change labels Mar 16, 2026
@copy-pr-bot

copy-pr-bot Bot commented Mar 16, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added Python Affects Python cuDF API. cudf.pandas Issues specific to cudf.pandas labels Mar 16, 2026
@GPUtester GPUtester moved this to In Progress in cuDF Python Mar 16, 2026
@vyasr
vyasr force-pushed the feat/cow_default branch from 09a0c7b to 1c4481a Compare March 16, 2026 23:08
@vyasr

vyasr commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 1c4481a

@vyasr vyasr changed the title Make copy-on-write the default for pandas3#21800 Make copy-on-write the default for pandas3 Mar 16, 2026

@mroeschke mroeschke left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice! In a follow up (not urgent for pandas 3 compat), I bet there are more places where we can call copy(deep=False) when coping column objects or in public APIs

@vyasr
vyasr merged commit 6d80219 into NVIDIA:pandas3 Mar 16, 2026
32 of 33 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in cuDF Python Mar 16, 2026
@vyasr
vyasr deleted the feat/cow_default branch March 16, 2026 23:26
@vyasr vyasr mentioned this pull request Mar 20, 2026
3 tasks
galipremsagar pushed a commit that referenced this pull request Mar 23, 2026
## Description
<!-- Provide a standalone description of changes in this PR. -->
<!-- Reference any issues closed by this PR with "closes #1234". -->
<!-- Note: The pull request title will be included in the CHANGELOG. -->
I missed this in #21801.

## Checklist
- [x] I am familiar with the [Contributing
Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md).
- [x] New or existing tests cover these changes.
- [x] The documentation is up to date with these changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change cudf.pandas Issues specific to cudf.pandas improvement Improvement / enhancement to an existing function Python Affects Python cuDF API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants