Github user aledsage commented on the issue:

    https://github.com/apache/brooklyn-server/pull/616
  
    @sjcorbett oops, I didn't realise it was in use. In hindsight, it's 
understandable that power-users in downstream projects would implement this so 
they can get the exact (configuration of) `BlobStoreContext` that they need.
    
    Other than the hassle of updating your impl, do the changes here make 
things particularly difficult for you?
    
    
    ---
    I previously had in my head that we'd really only have two impls: the 
`BlobStoreContextFactoryImpl` and a stub test impl for unit testing (which 
isn't written yet).
    
    That's how `ComputeServiceRegistry` is used, I believe (but who knows what 
folk have done in their downstream projects!).
    
    I hope that `BlobStoreContextFactoryImpl` can become configurable enough 
that folk don't need to create their own implementations, but maybe that's 
unrealistic.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to