rohangarg commented on PR #12948:
URL: https://github.com/apache/druid/pull/12948#issuecomment-1226792775

   The example only allows one binding per namespace and prefix. If more than 
one binding is tried for the same namespace, the guice injector will throw a 
binding error.
   BTW, the current example works on simple `Provider` interface where the 
instance creator doesn't accept the `property_prefix` to create objects with 
different properties at runtime. For such case where the provider is meant to 
be more like a factory, we'd could try to use `FactoryProvider` from guice or 
do a semantic implementation of factory using `Provider` ourselves.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to