-
Notifications
You must be signed in to change notification settings - Fork 463
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
docs: add aux file v2 RFC #9115
Conversation
Signed-off-by: Alex Chi Z <chi@neon.tech>
8b655e3
to
63fc656
Compare
I hope I've covered all key design decisions throughout the process, if not, I can add more to the RFC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good read!
I think there's no shame in pointing out at the beginning of the doc that this is a retro-active RFC.
Also, point out in Motivation that this RFC should help understand the migration code (aux_v2_migration.git
)
Signed-off-by: Alex Chi Z <chi@neon.tech>
5013 tests run: 4855 passed, 0 failed, 158 skipped (full report)Flaky tests (7)Postgres 17
Postgres 15
Postgres 14
Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
9726399 at 2024-09-30T21:57:57.249Z :recycle: |
Signed-off-by: Alex Chi Z <chi@neon.tech>
Signed-off-by: Alex Chi Z <chi@neon.tech>
Problem
aux v2 migration is near the end and I rewrote the RFC based on what I proposed (several months before...) and what I actually implemented.
Summary of changes
Add aux file v2 RFC
Checklist before requesting a review
Checklist before merging