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
Only write ref key once when writing with prune_previous_versions #1560
Conversation
… pytest continues to run indefinitely even after the tests have passed.
PR #1355 also added a
overwrite_symbol_tree is used in:
recover_deleted is only used in tests remove_and_rewrite_version_keys is unused scan_and_rewrite is used in |
…e symbol ref key. Usage: write_entry_to_storage |_ compact_and_remove_deleted_indexes (unused, no impact) |_ writes the symbol ref a second time |_ overwrite_symbol_tree |_ writes the symbol ref a second time |_ rewrite_entry |_ recover_deleted |_ writes the symbol ref a second time |_ remove_and_rewrite_version_keys |_ writes the symbol ref a second time |_ scan_and_rewrite |_ writes the symbol ref a second time overwrite_symbol_tree is used in: initial sync process in replication (no impact really, we don't expect stuff to be readable during the initial sync) fix_symbol_trees (unused) recover_deleted is only used in tests remove_and_rewrite_version_keys is unused scan_and_rewrite is used in fix_ref_key which is unused Testing limited as most call sites are unused. I verified that replication still works correctly after this change.
build_tooling/parallel_test.sh
Outdated
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.
These changes are just formatting.
…'t hang the CI suite for 6 hours...
) #### What does this implement or fix? We should only write the version ref key once when we write with `prune_previous_versions=True`. Currently we are writing it twice - once after we write the tombstone all and once when we write the new version. This means that there is a period of time where the symbol is unreadable. This was fixed a while ago with PR #1104 but regressed with PR #1355.
What does this implement or fix?
We should only write the version ref key once when we write with
prune_previous_versions=True
. Currently we are writing it twice - once after we write the tombstone all and once when we write the new version. This means that there is a period of time where the symbol is unreadable.This was fixed a while ago with PR #1104 but regressed with PR #1355.
Any other comments?
Checklist
Checklist for code changes...