nsivabalan commented on issue #16809:
URL: https://github.com/apache/hudi/issues/16809#issuecomment-4795641504

   Looks like this is already resolved on master.
   
   The three tests added by 
[HUDI-8775](https://github.com/apache/hudi/pull/12558) are active in 
`TestExpressionIndex.scala` and not commented out:
   
   - `Test expression index partition pruning with partition stats` (line 1219)
   - `Test expression index pruning after update with partition stats` (line 
1317)
   - `Test expression index pruning after delete with partition stats` (line 
1449)
   
   In the merged version of #12707 (commit `474942304ca6`), the failing 
assertion was fixed in-place by scoping the partition-stats validation query to 
the rider keys (`... AND key IN ('$riderCalifornia', '$riderTexas')`), which is 
needed once partition stats is built for all columns by default. PR #13546 
(`b479f12285fd`) later just renamed the JUnit `@Test` methods to scala-test 
`test("...")` form.
   
   Proposing to close HUDI-8919 / #16809 unless someone is still seeing a 
failure on master.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to