search

apache / pinot

Apache Pinot - A realtime distributed OLAP datastore
https://pinot.apache.org/
Apache License 2.0
5.54k stars 1.3k forks source link

Merge `RecordEnricher` into `RecordTransformer` #13049

Open Jackie-Jiang opened 7 months ago

Jackie-Jiang commented 7 months ago

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

deepthi912 commented 7 months ago

Looking into it.

aadilkhalifa commented 4 months ago

@deepthi912 Are you working on this?

deepthi912 commented 4 months ago

@aadilkhalifa Not currently working on it but you can work if you are willing to!