Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unify RowId and time columns #9082

Open
emilk opened this issue Feb 19, 2025 · 0 comments
Open

Unify RowId and time columns #9082

emilk opened this issue Feb 19, 2025 · 0 comments
Labels
🔩 data model Sorbet

Comments

@emilk
Copy link
Member

emilk commented Feb 19, 2025

We currently treat the RowId column as a special snowflake. However, it would be nice if we treated it the same as our time columns, unifying them under an index column type.

The first advantage is that the dataframe API would allow users to select (or not select) the row id column.

In the limit, this would be part of an effort to expand the types of indices we allow beyond i64 (which we use for time/sequence).
@emilk emilk added the 🔩 data model Sorbet label Feb 19, 2025
This was referenced Feb 19, 2025
Open
Open
Merged
emilk added a commit that referenced this issue Feb 19, 2025
@emilk
Make IndexColumnDescriptor members private (#9070)
920de91 
### Related
* Sibling PR: rerun-io/dataplatform#259
* Part of #8744
* Part of #9082

### What
A few steps on the road to unifying timelines and RowId


### TODO
* [x] dataplatform PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🔩 data model Sorbet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@emilk