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.

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

Automatically generate partial update APIs for eager archetypes #8647

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

teh-cmc
Copy link
Member

@teh-cmc teh-cmc commented Jan 10, 2025

This extends the codegen of the attr.rust.archetype_eager attribute so that it automatically generates partial updates APIs for all eager archetypes.

Copy link

github-actions bot commented Jan 10, 2025

Web viewer failed to build.

| Result | Commit | Link | Manifest |
| ------ | ------- | ----- |
| ❌ | | https://rerun.io/viewer/pr/8647 | +nightly +main |

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

@teh-cmc teh-cmc marked this pull request as ready for review January 13, 2025 08:05
@teh-cmc teh-cmc force-pushed the cmc/eager_partial_3_eager_archetypes branch from 15b27bd to 159b53f Compare January 13, 2025 08:22
Base automatically changed from cmc/eager_partial_3_eager_archetypes to main January 13, 2025 08:35
@teh-cmc teh-cmc force-pushed the cmc/eager_partial_4_partial_on_eager branch from a1331e1 to d5bc79c Compare January 13, 2025 08:36
@teh-cmc teh-cmc removed the do-not-merge Do not merge this PR label Jan 13, 2025
@teh-cmc teh-cmc merged commit 1b9e510 into main Jan 13, 2025
28 of 29 checks passed
@teh-cmc teh-cmc deleted the cmc/eager_partial_4_partial_on_eager branch January 13, 2025 08:37
teh-cmc added a commit that referenced this pull request Jan 13, 2025
Store-side compaction has election heuristics that can yield different
results depending on the order in which the chunks arrive.
This can make some cross-language roundtrip tests flaky.

This PR makes sure to disabled store-side compaction when running RRD
comparison tests.

* DNM: requires #8647
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant