apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tushargosavi <...@git.apache.org>
Subject [GitHub] apex-core pull request #361: APEXCORE-496 make operator name available to St...
Date Wed, 10 Aug 2016 11:45:56 GMT
Github user tushargosavi commented on a diff in the pull request:

    https://github.com/apache/apex-core/pull/361#discussion_r74229007
  
    --- Diff: api/src/main/java/com/datatorrent/api/StatsListener.java ---
    @@ -115,6 +115,28 @@
         List<OperatorResponse> getOperatorResponse();
       }
     
    +  /**
    +   * An interface to the DAG. Stats listener can get information about
    +   * operator or other elements in the DAG through this interface. currerntly
    +   * we only provide method to extract the operator name based on the physical
    +   * id of the operator. In future more methods can be added.
    +   *
    +   */
    --- End diff --
    
    exposing all information may expose internal implementation of the DAG. we can only have
dependency on apex-api. Can you suggest which information can be provided to stat listener
without adding dependency on the apex-engine. I can think of following
    - name
    - number of partitions for self.
    - aggregated stats at logical operator level.
    This list could be expanded in future.
    
    This jira puts the required infrastructure without breaking backward compatibility.


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