Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

emilk
Copy link
Member

@emilk emilk commented Jun 24, 2025

Related

What

NOTE! This is a 0.23.4 release branch, as I want to get these fixes out ASAP.

Problem 1: sorbet meta key name confusion

At some point we renamed the keys of our meta-data, and forgot to update our code:

  • rerun.archetype vs rerun.archetype_name
  • rerun.archetype_field vs rerun.archetype_field_name

Problem 2: not writing new meta keys on migrate

Our migration code erroneously would keep the old metadata of all our fields.

The new code in this PR not only fixes this, but also retains all non-rerun metadata, because it's the right thing to do.

TODO

  • Test that migrated files work on main
  • Port the second commit to main
  • Port changelog to main

@emilk emilk added the do-not-merge Do not merge this PR label Jun 24, 2025
Copy link

github-actions bot commented Jun 24, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
f18365d https://rerun.io/viewer/pr/10350 +nightly +main

Note: This comment is updated whenever you push a commit.

@grtlr grtlr self-requested a review June 25, 2025 10:25
@emilk emilk marked this pull request as ready for review June 25, 2025 10:32
@emilk emilk changed the title Fix migration of old 0.22 rrd files Release 0.23.4: Fix migration of old 0.22 rrd files Jun 25, 2025
Copy link
Member

@grtlr grtlr left a comment

Choose a reason for hiding this comment

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

Ah nice, how subtle!

debug_assert_eq!(new_field.data_type(), column.data_type());

let mut metadata = old_field.metadata().clone();
metadata.extend(new_field.metadata().clone()); // overwrite old with new
Copy link
Member

Choose a reason for hiding this comment

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

Would that work in a setting where we change keys as part of our migration? Because that currently happens on main.

Copy link
Member Author

Choose a reason for hiding this comment

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

It would, if we would call rewire_tagged_components at the top of SorbetBatch::try_from_record_batch… unfortunately we call it only in ChunkBatch::try_from, which I think is a separate bug (we should always rewire tagged components)

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll make sure this is correct when I cherry-pick the commit into main

Copy link
Member

@jprochazk jprochazk left a comment

Choose a reason for hiding this comment

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

@emilk
This PR needs to be based off of release-0.23.4-alpha.2 😬 it contains some important changes to the build process, but it was never fully released as 0.23.4

I think those changes can also just be cherry-picked in case you don't want to rebase

https://github.com/rerun-io/rerun/tree/release-0.23.4-alpha.2

@emilk
Copy link
Member Author

emilk commented Jun 25, 2025

Good catch - I'll rebase

Copy link

github-actions bot commented Jun 25, 2025

Latest documentation preview deployed successfully.

Result Commit Link
f18365d https://landing-k3rfnpmqj-rerun.vercel.app/docs

Note: This comment is updated whenever you push a commit.

@emilk
Copy link
Member Author

emilk commented Jun 25, 2025

@jprochazk jprochazk dismissed their stale review June 25, 2025 11:58

addressed

@emilk
Copy link
Member Author

emilk commented Jun 25, 2025

Release job failed:

image

Comment on lines +16 to 33
arrow-devel \
make automake gcc gcc-c++ kernel-devel \
cmake \
curl \
git \
git-lfs \
libatk-bridge2.0 \
libfontconfig1-dev \
libfreetype6-dev \
libglib2.0-dev \
libgtk-3-dev \
libssl-dev \
libxcb-render0-dev \
libxcb-shape0-dev \
libxcb-xfixes0-dev \
libxkbcommon-dev \
lsb-release \
atk at-spi2-atk \
fontconfig-devel \
freetype-devel \
glib2-devel \
gtk3-devel \
openssl-devel \
xcb-util-renderutil-devel \
xcb-util-devel \
xcb-util-wm-devel \
libxkbcommon-devel \
python3-pip \
software-properties-common \
sudo; \
Copy link
Member Author

Choose a reason for hiding this comment

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

nit: it would be easier to merge changes to this list if it was sorted

Copy link
Member Author

Choose a reason for hiding this comment

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

Also a bit weird that some things are grouped (atk, make/automake) and others not (xcb) 🤔

@emilk emilk added the ⛴ release Related to shipping or publishing label Jun 25, 2025
@rerun-bot
Copy link
Collaborator

Version 0.23.4-rc.2 published successfully.

artifact install
web app
wheels pip install rerun-sdk==0.23.4-rc.2
crates cargo install [email protected] --locked
npm npm install @rerun-io/[email protected]
docs
py docs
rs docs
cpp_sdk zip

@rerun-bot
Copy link
Collaborator

Version 0.23.4 published successfully.

artifact install
web app
wheels pip install rerun-sdk==0.23.4
crates cargo install [email protected] --locked
npm npm install @rerun-io/[email protected]
docs
py docs
rs docs
cpp_sdk zip

@emilk emilk mentioned this pull request Jun 26, 2025
@emilk emilk closed this Jun 26, 2025
emilk added a commit that referenced this pull request Jun 26, 2025
emilk added a commit that referenced this pull request Jun 27, 2025
)

### Related
* Part of #10273
* Cherry-picked from #10350

### What
Ensure that all the Rerun-specific metadata that is added to a record
batch actually makes it into the record batch

* [x] full-check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Do not merge this PR ⛴ release Related to shipping or publishing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants