This repository has been archived by the owner on Jul 24, 2024. It is now read-only.
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.
restore: merge small ranges #578
restore: merge small ranges #578
Changes from 5 commits
fa4d3a1
f1298e2
e9c6795
57ecf92
9923efc
f2a1d40
8998c1d
410de98
0f83629
22d59a6
5c3f29f
4ff72de
9d88692
6d1c5f5
a5955c3
7218d0a
da86c97
4c28fab
42800ae
0e94162
8e19d4d
a241278
03bef3c
047d50a
6adfcee
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Can we merge index range and row key range with same table id here?
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.
It's ok as long as they belong to the same table.
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.
If we merge record key and index key into one file. we need to find the correct rewrite rule.
br/pkg/restore/import.go
Line 212 in c344d1e
but default rewrite rule starts with record prefix
br/pkg/restore/util.go
Line 73 in c344d1e
and with this rule sst_importer cannot rewrite index key correctly.
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.
You are right, we need to separate ranges by index ID too.