Codecov Report

Merging #9338 (8c82c4d) into master (2f19683) will decrease coverage by 45.02%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #9338       +/-   ##
=============================================
- Coverage     69.88%   24.85%   -45.03%     
+ Complexity     5008       53     -4955     
=============================================
  Files          1854     1864       +10     
  Lines         98871    99508      +637     
  Branches      15043    15156      +113     
=============================================
- Hits          69094    24732    -44362     
- Misses        24904    72275    +47371     
+ Partials       4873     2501     -2372     

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Suggest adding some tests to test end to end query execution by using this function in different parts of the query -- example filter, select etc

There is also LogicalOperatorTransformFunction. Should we implement this as part of that or a subclass of it ? It supports AND, OR and we want to add NOT

Just curious - how do you plan on using this as part of post aggregation ? Share a query example ? Would be good to have e2e query test

I didn't see the previous PR #9330 until now. Now it makes sense why we need this PR to handle post aggregation