Robert Bradshaw created BEAM-3046:
-------------------------------------

             Summary: De-javafy the Python Metrics API
                 Key: BEAM-3046
                 URL: https://issues.apache.org/jira/browse/BEAM-3046
             Project: Beam
          Issue Type: Bug
          Components: sdk-py-core
            Reporter: Robert Bradshaw
            Assignee: Pablo Estrada


For example, though Java forces one to put all stand-alone functions as static 
methods on a class (like Metrics) in Python one would just have standard 
module-level functions. 

Also, the MetricsFilter builder pattern is very Java-esque (and verbose). In 
Python, one would simply let result.metrics() take keyword arguments. 

We should seriously consider aggregating rather than returning lists of 
results, at least by default. E.g. in 
https://github.com/apache/beam/blob/master/sdks/python/apache_beam/examples/wordcount.py
 it assumes that there's no other metric with the given name in any 
subtransform (otherwise it may deterministically grab the "wrong" one.)



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to