apache/beam

python typehints: reduce the number of runtime-type-check wrappers

Ouverte

#20 226 ouverte le 4 juin 2022

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Java (4 097 forks)batch import
P3bugcoregood first issuepythontypes

Métriques du dépôt

Stars
 (7 313 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

The --runtime_type_check flag is considered to be a performance penalty. One of the possible slowdowns is the wrapper around DoFn.process(), which adds an extra call to the wrapped process() method. Currently, there are 2 wrappers applied: OutputCheckWrapperDoFn and TypeCheckWrapperDoFn. These could be merged into one to possibly save on overhead.

Imported from Jira BEAM-9489. Original Jira may contain additional context. Reported by: udim.

Guide contributeur

Direction de recherche
Examinez les classes existantes OutputCheckWrapperDoFn et TypeCheckWrapperDoFn, identifiez leurs responsabilites et concevez un wrapper unifie qui combine les deux fonctionnalites tout en minimisant le surcout.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonApache Beam
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55