-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Closed
Labels
Storage:RemoteenhancementEnhancement or improvement to existing feature or requestEnhancement or improvement to existing feature or requestv2.14.0
Description
Is your feature request related to a problem? Please describe
We have introduced a new hash prefix strategy for remote store in #12567. With the new prefix strategy, we also need to upload the index paths. For new indexes, we have a running PR - #13150. However, based on my current understanding, for migration, this would not work since the current hook is only for new indexes.
Describe the solution you'd like
The hook that is beforeNewIndexUpload in IndexMetadataUploadListener needs to be updated or changed to a model where we would have the list of index metadata that needs to be updated along with their previous state of index metadata. We can then change the logic where it checks that the remoteCustomData is not present in previous state and present now. In such cases, we can upload the index path.
Related component
Storage:Remote
Describe alternatives you've considered
NA
Additional context
NA
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Storage:RemoteenhancementEnhancement or improvement to existing feature or requestEnhancement or improvement to existing feature or requestv2.14.0
Type
Projects
Status
✅ Done
Status
Planned work items