brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ahgittin <...@git.apache.org>
Subject [GitHub] incubator-brooklyn pull request: Introduce relationships between b...
Date Tue, 27 Oct 2015 12:02:58 GMT
Github user ahgittin commented on the pull request:

    https://github.com/apache/incubator-brooklyn/pull/987#issuecomment-151469204
  
    I've added a comment on the double dispatch via `EntityRelations` statics:
    
    ```
    /** This abstract impl delegates to {@link EntityRelations} for all changes, routing through
a local backing store.
     * This allows us to make the changes in both directions simultaneously when a relationship
is bi-directional,
     * and should facilitate changing a backing datastore or remote instances when that is
supported.
     * <p>
     * Currently it can be implemented without it, simplifying things a bit (avoiding the
double dispatch)
     * apart from the {@link #add(Relationship, BrooklynObject)} method triggering the reverse
addition
     * if it isn't already present. TBD which is better (and the internal call to get the
backing store is 
     * marked as Beta). */
    ```
    
    WDYT?


---
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