Skip to content

refactor(static-metadata): undo migration of feature_states to static metadata #1146

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

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

glevco
Copy link
Contributor

@glevco glevco commented Oct 7, 2024

Motivation

A couple PRs were previously merged in preparation for migrating the feature_states attribute from "normal" metadata to static metadata (#1016, #1017). As this migration has been postponed, this PR undoes some of those previous changes such as bringing back removed tests.

Effectively, it cancels the unmerged #1018. The reason for cancelling it is that it changes the Feature Activation algorithm a bit, requiring careful review, and we don't want to invest the time in it right now. Instead, we want to merge Feature Activation for Transactions (#933) as soon as possible to begin testing it, so we're cancelling #1018 which was a dependency to it #933.

Acceptance Criteria

This PR contains 2 main changes:

  1. A pure git revert of refactor(metadata): pre-refactors for migrating feature states to static metadata [part 4/12] #1017, bringing back removed test assertions (in 1aad3cc).
  2. A partial revert of refactor(metadata): pre-refactors for migrating feature states to static metadata [part 3/12] #1016, bringing back a single removed test (in af8cbcf).

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

Copy link

github-actions bot commented Oct 7, 2024

🐰 Bencher Report

Branchrefactor/static-metadata/undo-feature-states
Testbedubuntu-22.04

⚠️ WARNING: The following Measure does not have a Threshold. Without a Threshold, no Alerts will ever be generated!

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds CLI flag.

Click to view all benchmark results
BenchmarkLatencynanoseconds (ns)
sync-v2 (up to 20000 blocks)📈 view plot
⚠️ NO THRESHOLD
101,450,350,593.08
🐰 View full continuous benchmarking report in Bencher

Copy link

codecov bot commented Oct 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.37%. Comparing base (daf12e7) to head (2d27cd1).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1146      +/-   ##
==========================================
- Coverage   84.51%   84.37%   -0.15%     
==========================================
  Files         317      317              
  Lines       24389    24389              
  Branches     3709     3709              
==========================================
- Hits        20612    20577      -35     
- Misses       3066     3089      +23     
- Partials      711      723      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glevco glevco force-pushed the refactor/static-metadata/undo-feature-states branch 2 times, most recently from e31b9dc to 0e58201 Compare October 8, 2024 14:53
@glevco glevco changed the title undo feature states refactor(static-metadata): undo migration of feature_states to static metadata Oct 8, 2024
@glevco glevco marked this pull request as ready for review October 8, 2024 15:33
jansegre
jansegre previously approved these changes Oct 8, 2024
@glevco glevco force-pushed the refactor/static-metadata/undo-feature-states branch from 0e58201 to b1d9f54 Compare October 10, 2024 19:42
@glevco glevco force-pushed the refactor/static-metadata/undo-feature-states branch from b1d9f54 to f179ea7 Compare October 10, 2024 19:45
@glevco glevco force-pushed the refactor/static-metadata/undo-feature-states branch from f179ea7 to b8584e2 Compare October 11, 2024 16:49
@glevco glevco force-pushed the refactor/static-metadata/undo-feature-states branch from b8584e2 to 2d27cd1 Compare October 11, 2024 16:53
@glevco glevco merged commit 6a4c562 into master Oct 14, 2024
13 checks passed
@glevco glevco deleted the refactor/static-metadata/undo-feature-states branch October 14, 2024 03:37
@jansegre jansegre mentioned this pull request Oct 14, 2024
2 tasks
@jansegre jansegre mentioned this pull request Dec 11, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants