apurtell commented on PR #2523:
URL: https://github.com/apache/phoenix/pull/2523#issuecomment-4694067445

   ## Test Results
   
   ### Unit tests — PASS
   
   | Test class | Tests | Failures | Errors | Skipped |
   | --- | --- | --- | --- | --- |
   | `QueryOptimizerTest` (modified — +3 new tests) | 49 | 0 | 0 | 1 |
   | `QueryCompilerTest` | 215 | 0 | 0 | 2 |
   | `ExplainPlanTest` | 85 | 0 | 0 | 0 |
   | `TenantSpecificViewIndexCompileTest` | 6 | 0 | 0 | 0 |
   | `QueryPlanTest` | 4 | 0 | 0 | 0 |
   | `StatementHintsCompilationTest` | 4 | 0 | 0 | 0 |
   | **Total** | **363** | **0** | **0** | **3** |
   
   New tests added by the commit (all passing):
   - `testFunctionalIndexChosenRuleMatchesExpression` — chosen functional 
index's rule is overridden to `matches UPPER(NAME)`.
   - `testFunctionalIndexRejectedPathExpressionDoesNotMatch` — functional index 
that matched no query path expression is rejected with `path expression does 
not match`.
   - `testFunctionalIndexRejectedDoesNotCoverProjection` — functional index 
whose expression matched but which does not cover the projection keeps the 
`does not cover projection` rejection.
   
   ### Integration tests (`*IT`) — PASS
   
   | Test class | Tests | Failures | Errors | Skipped |
   | --- | --- | --- | --- | --- |
   | `IndexUsageIT` (functional indexes `s2||'_'||s3`, `k1/k2`) | 39 | 0 | 0 | 
0 |
   | `JsonFunctionsIT` (JSON path functional indexes) | 15 | 0 | 0 | 0 |
   | **Total** | **54** | **0** | **0** | **0** |
   
   Both IT classes exercise the changed optimizer code paths via real plan 
compilation against a minicluster and assert on `indexRule` / `indexRejected`.
   


-- 
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