apache/beam

De-javafy the Python Metrics API

Open

#18.597 aperta il 3 giu 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (4097 fork)batch import
P3community-onboardingcoregood first issueimprovementpython

Metriche repository

Star
 (7313 star)
Metriche merge PR
 (Merge medio 6g) (243 PR mergiate in 30 g)

Descrizione

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

Imported from Jira BEAM-3046. Original Jira may contain additional context. Reported by: robertwb.

Guida contributor

Direzione di ricerca
Analizza l'implementazione attuale dell'API Metrics di Python e identifica i metodi che possono essere convertiti da metodi statici in stile Java a funzioni a livello di modulo. Esamina il pattern builder MetricsFilter e sostituiscilo con argomenti keyword. Considera di aggregare i risultati per impostazione predefinita.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonApache Beam
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75