rbind + setkey slower than xts::rbind, which automatically indexes

Open
#873 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
r
Domain
data, performance

Research direction

Start with the linked Stack Overflow timings and the rbindlist and key behavior described in the issue. Determine how an additional key argument should produce sorted output during binding, using the reported xts::rbind comparison as the performance target. The issue is done when the behavior and performance are validated against that comparison.

Written by the indexing model from the issue text.

Description

[rbind|cbind|merge]list feature request

rbind.xts is implemented in C, and if the reindexing is done in the same step, then it does it more efficiently. In data.table, we've to bind first, then rearrange it.

rbindlist should then implement an additional key argument to return results in the sorted order already to get the performance on par with xts::rbind.

Timings and reproducible code on SO, as discussed with @arunsrinivasan

Dominant language
R
Stars
3.9k
Forks
1.1k
Avg merge
14h 4m
Merged PRs (30d)
4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Rdatatable/data.table

All issues in Rdatatable/data.table

Similar issues

More R issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.