-
Notifications
You must be signed in to change notification settings - Fork 506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - fix(stream_model): prevent collisions between namespaces #2539
Conversation
@@ -163,11 +163,13 @@ pub trait K8ExtendedSpec: Spec { | |||
|
|||
fn convert_from_k8( | |||
k8_obj: K8Obj<Self::K8Spec>, | |||
multi_namespace_context: bool, | |||
) -> Result<MetadataStoreObject<Self, K8MetaItem>, K8ConvertError<Self::K8Spec>>; | |||
} | |||
|
|||
pub fn default_convert_from_k8<S>( |
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.
I think that we should add some references to the way that we should use fluvio_stream_dispatcher::store::StoreContext
when using multiple namespaces for all the methods that have key: S::IndexKey
as a parameter. Maybe as docstrings?
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.
added docstring to default_convert_from_k8
bors r+ |
Co-authored-by: Nick Cardin <nick@cardin.email>
Pull request successfully merged into master. Build succeeded: |
Can you add more description to this PR? Or should this have an associated issue describing API changes would intendend |
This is probably not fix since this is not bug fix |
No description provided.