brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From neykov <...@git.apache.org>
Subject [GitHub] brooklyn-server issue #423: BROOKLYN-382: get-catalog performance improvemen...
Date Thu, 10 Nov 2016 10:34:00 GMT
Github user neykov commented on the issue:

    https://github.com/apache/brooklyn-server/pull/423
  
    Need to clear the **whole** cache on any addition/removal/update. Specs will change when
transient catalog item dependencies change.
    There are things outside of the control of the catalog that a spec depends on - like non-catalog
locations, type registry, adding bundles.
    Re-using the specs will cause objects to be shared between applications - something that
could lead to subtle problems (similar to the dsl memberSpeck locking).
    
    Having said that I believe `BrooklynCatalog.createSpec` is only used by the REST API so
not as bad as it sounds. But better move the logic to the REST API?
    
    The cache will speed up listing, but not adding items or deploying which could still take
considerable time. A much better fix would be to streamline our catalog parsing logic which
currently is way too complicated.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message