Skip to content

add shared guard for setkey#7769

Draft
ben-schwen wants to merge 1 commit into
masterfrom
setkey_shallow_copy
Draft

add shared guard for setkey#7769
ben-schwen wants to merge 1 commit into
masterfrom
setkey_shallow_copy

Conversation

@ben-schwen
Copy link
Copy Markdown
Member

Closes #5230

@ben-schwen ben-schwen requested review from MichaelChirico and aitap and removed request for MichaelChirico May 29, 2026 20:01
@ben-schwen ben-schwen marked this pull request as draft May 29, 2026 20:07
@github-actions
Copy link
Copy Markdown

  • HEAD=setkey_shallow_copy much slower for as.data.table.array improved in #7019
    Comparison Plot

Generated via commit 1d17aaa

Download link for the artifact containing the test results: ↓ atime-results.zip

Task Duration
R setup and installing dependencies 6 minutes and 27 seconds
Installing different package versions 12 minutes and 13 seconds
Running and plotting the test cases 5 minutes and 23 seconds

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DT1 row selection broken after DT2 = DT1[!b%in%x], setkey(DT2,a)

1 participant