Hi Team,
Please review this <PR> to add the functi...
# generals
Suraj Goel
09/13/2025, 2:07 PMHi Team,
Please review this PR to add the functionality of kafka header based filtering.
Thanks
k
Karan Kumar
09/17/2025, 3:32 PMThis seems cool. We will get to it soon.
Karan Kumar
09/17/2025, 3:33 PMCould you please add a embedded test for the PR.
✅ 1
Karan Kumar
09/17/2025, 3:33 PMThat would help in making this feature future proof.
s
Suraj Goel
09/22/2025, 7:31 AM@Karan Kumar, I have added an embedded test.
Please take a look. TIA!
Suraj Goel
09/29/2025, 7:00 AM@Karan Kumar,
gentle reminder to get this PR reviewed.
Suraj Goel
10/06/2025, 7:42 AMbumping this up!
k
Karan Kumar
10/07/2025, 3:35 PMHey I can probably get to it next week.
👍 1
s
Suraj Goel
10/17/2025, 4:59 AMHey @Karan Kumar,
We have done load testing for this feature as well and wanted to deploy it on production clusters.
Could you please take a look at this PR.
k
Karan Kumar
10/21/2025, 12:46 PMLeft an initial review.
Karan Kumar
10/21/2025, 12:46 PMThanks for the patience.
s
Suraj Goel
10/21/2025, 1:00 PMThanks @Karan Kumar,
replied to your comments. Let me know your thoughts.
Suraj Goel
10/23/2025, 5:00 AMHey, Addressed your comments to rename the config back to
headerBasedFilterConfigSuraj Goel
10/28/2025, 4:08 AM@Frank Chen, @Karan Kumar,
I have addressed most of the comments.
Could you please re-review. Thanks..
Suraj Goel
11/06/2025, 10:47 AMGentle reminder on this!
Suraj Goel
11/20/2025, 8:24 AMGentle reminder on this!
k
Karan Kumar
11/26/2025, 2:44 PMThanks for the reminder. Would plan to merge this post thanksgiving week
👍 1
s
Suraj Goel
12/11/2025, 4:05 PMFYI, We have deployed this feature on our prod clusters and ingestion pipeline is running smoothly with reduced CPU usage on indexers.