feat(catalog): Implement MemoryCatalog's table update/ commit path #1405
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a copy of #1290. Reopened here for reference.
Which issue does this PR close?
Relates to #700
This is a first attempt at implementing the commit path for the
MemoryCatalog
. Happy about any feedback!What changes are included in this PR?
This PR implements the
MemoryCatalog::update_table
method. It also adds aMetadataLocation
struct to avoid some re-validations of the location format.I'm very open to removing the
MetadataLocation
or moving it's addition to a separate PR.Are these changes tested?
So far, I've included unit tests to verify
MemoryCatalog
can be observed in the updated tableMetadataLocation