Skip to content

Conversation

Bdegraaf1234
Copy link
Contributor

Release PR

blootsvoets and others added 30 commits December 5, 2022 10:27
This is especially useful for excluding projectId and userId, since
those are already included in the directory structure.
- TargetStorage now takes the first part of the path as the target bucket name.
- Gave RecordPathFactory some more hardcoded config.
- Updated dependencies.
Excluding can be done per topic or for all topics
Before every restructuring or cleaning operation, the index is updated.
The StorageIndex can be updated with a configurable sync time to make a
full sync, otherwise it just updates directories that have files in them.
A separate sync time can be set to also scan empty directories.
The first implementation is only a InMemoryStorageIndex. For
very large datasets, a file-based index might be needed.
Added a StorageIndex for the source storage to reduce LIST calls
security updates and updated maintainers
@Bdegraaf1234 Bdegraaf1234 self-assigned this Aug 19, 2024
Copy link
Contributor

@pvannierop pvannierop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (but I have little experience with this service to make a good judgement, so I will rely on Bastiaan's expertise here)

As suggested by Dependabot.
@pvannierop pvannierop merged commit 3d79613 into main Sep 10, 2024
5 checks passed
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.

3 participants