[GitHub] incubator-quarks pull request: [QUARKS-131] [REVIEW] supply an ali...

[vdogaru <gi...@git.apache.org>]

Github user vdogaru commented on a diff in the pull request:

    https://github.com/apache/incubator-quarks/pull/86#discussion_r60467472
  
    --- Diff: api/graph/src/main/java/quarks/graph/Connector.java ---
    @@ -115,5 +115,27 @@ Licensed to the Apache Software Foundation (ASF) under one
          * 
          * @return set of tag values.
          */
    -    Set<String> getTags(); 
    +    Set<String> getTags();
    +    
    +    /**
    +     * Set the alias for the connector.
    +     * <p>
    +     * The alias must be unique within the topology.
    +     * The alias may be used in various contexts:
    +     * <ul>
    +     * <li>Runtime control services for the connector are registered with this alias.</li>
    --- End diff --
    
    I'm a bit confused by "Runtime control services for the connector".  At the high level, a control service provides an interface for controlling the stream and is registered with the TStream's alias.



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