-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Closed
Labels
cleanupCode cleanup or removal of dead codeCode cleanup or removal of dead codehelp wantedExtra attention is needed — open for contributionsExtra attention is needed — open for contributionsrefactorCode restructuring without changing behaviorCode restructuring without changing behavior
Description
RecordEnricher was introduced in #12243, which is very similar to RecordTransformer. It can cause confusion to maintain both of the abstractions because there is no clear boundary when to use which one.
We should merge RecordEnricher into RecordTransformer, and make the existing enrichers implement RecordTransformer interface. We might need to modify RecordTransformer a little bit to serve the purpose of RecordEnricher
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
cleanupCode cleanup or removal of dead codeCode cleanup or removal of dead codehelp wantedExtra attention is needed — open for contributionsExtra attention is needed — open for contributionsrefactorCode restructuring without changing behaviorCode restructuring without changing behavior