Re: [I] Fix Expression index and partition pruning [hudi]

["nsivabalan (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org