Skip to content

storage: add Prefixed wrapper for namespace scoping#67

Merged
bigbes merged 1 commit intomasterfrom
bigbes/tntp-7587-prefixed-storage-wrapper
Apr 29, 2026
Merged

storage: add Prefixed wrapper for namespace scoping#67
bigbes merged 1 commit intomasterfrom
bigbes/tntp-7587-prefixed-storage-wrapper

Conversation

@bigbes
Copy link
Copy Markdown
Collaborator

@bigbes bigbes commented Apr 29, 2026

Add storage.Prefixed Storage that scopes every operation, predicate, Range, and Watch call under the given namespace prefix. Nested Prefixed wrappers are flattened at construction.

Copy link
Copy Markdown
Collaborator

@Mockird31 Mockird31 left a comment

Choose a reason for hiding this comment

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

Please add an entry to CHANGELOG.md about this patch.

@bigbes bigbes force-pushed the bigbes/tntp-7587-prefixed-storage-wrapper branch from f67dd63 to 6ee7cd1 Compare April 29, 2026 10:05
@bigbes bigbes requested a review from Mockird31 April 29, 2026 10:05
@bigbes
Copy link
Copy Markdown
Collaborator Author

bigbes commented Apr 29, 2026

@Mockird31 fixed, ty!

@bigbes bigbes force-pushed the bigbes/tntp-7587-prefixed-storage-wrapper branch from 6ee7cd1 to 6d22d18 Compare April 29, 2026 11:32
Add storage.Prefixed(prefix, inner Storage) Storage that scopes every
operation, predicate, Range, and Watch call under the given namespace
prefix. Nested Prefixed wrappers are flattened at construction.

Part of TNTP-7587
@bigbes bigbes force-pushed the bigbes/tntp-7587-prefixed-storage-wrapper branch from 6d22d18 to 28685d2 Compare April 29, 2026 11:44
Comment thread prefix.go
@bigbes bigbes merged commit 7fad8ba into master Apr 29, 2026
17 checks passed
@bigbes bigbes deleted the bigbes/tntp-7587-prefixed-storage-wrapper branch April 29, 2026 13:24
@bigbes bigbes mentioned this pull request May 5, 2026
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