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

fix: stop storing TimelineMetadata in index_part.json as bytes #7699

Draft
wants to merge 9 commits into
base: joonas/legacy_metadata_as_json_in_future
Choose a base branch
from

Conversation

koivunej
Copy link
Contributor

@koivunej koivunej commented May 10, 2024

Problem

We've stored metadata as bytes within the index_part.json for since long fixed reasons. #7693 added support for reading out normal json serialization of the TimelineMetadata.

Summary of changes

Only write TimelineMetadata as json for going forward, keeping the backwards compatibility to reading the metadata as bytes.

In documentation and naming, an effort is made to add enough warning signs around TimelineMetadata so that it will receive no changes in the future. We can just add those fields to IndexPart directly instead. Changing TimelineMetadata now carries all of the backward compatibility risks without offering any benefits over changing IndexPart, where we do have a clear strategy (soft json (de)serialization).

Builds upon #7693.

Copy link

3024 tests run: 2885 passed, 6 failed, 133 skipped (full report)


Failures on Postgres 16

Failures on Postgres 15

Failures on Postgres 14

# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_forward_compatibility[release-pg14] or test_forward_compatibility[debug-pg14] or test_forward_compatibility[release-pg15] or test_forward_compatibility[debug-pg15] or test_forward_compatibility[release-pg16] or test_forward_compatibility[debug-pg16]"
Flaky tests (2)

Postgres 16

  • test_pageserver_lsn_wait_error_safekeeper_stop: debug

Postgres 15

  • test_partial_evict_tenant[relative_equal]: release

Test coverage report is not available

The comment gets automatically updated with the latest test results
48174cd at 2024-05-10T14:15:50.483Z :recycle:

@koivunej
Copy link
Contributor Author

Failures on Postgres 16

Failures on Postgres 15

Failures on Postgres 14

These were expected 🎉.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant