apurtell commented on PR #2503:
URL: https://github.com/apache/phoenix/pull/2503#issuecomment-4656739476
## Test results
### Scope
`git show --stat 7c6609046d` ("PHOENIX-7881 Refactor UTs and ITs to assert
on ExplainPlanAttributes") touches **143 files (+4,819 / −5,241)**: 20
production (`src/main`), 112 integration (`src/it`), 11 unit (`src/test`).
The only added runtime behavior is **attribute population**. Text rendering
is byte-for-byte unchanged.
### Surefire (unit tests)
```text
mvn -pl phoenix-core test -DforkCount=6 -Djacoco.skip=true \
-Dtest='ExplainPlanTest,JoinQueryCompilerTest,ParallelIteratorsSplitTest,PostIndexDDLCompilerTest,QueryCompilerTest,QueryOptimizerTest,QueryPlanTest,StatementHintsCompilationTest,TenantSpecificViewIndexCompileTest'
```
| Test class | Run | Failures | Errors | Skipped |
|---|---:|---:|---:|---:|
| `query.explain.ExplainPlanTest` (text corpus) | 59 | 0 | 0 | 0 |
| `compile.QueryCompilerTest` | 215 | 0 | 0 | 2 |
| `compile.QueryOptimizerTest` | 49 | 0 | 0 | 1 |
| `compile.TenantSpecificViewIndexCompileTest` | 6 | 0 | 0 | 0 |
| `compile.StatementHintsCompilationTest` | 4 | 0 | 0 | 0 |
| `query.QueryPlanTest` | 4 | 0 | 0 | 0 |
| `compile.PostIndexDDLCompilerTest` | 3 | 0 | 0 | 0 |
| `compile.JoinQueryCompilerTest` | 2 | 0 | 0 | 0 |
| `query.ParallelIteratorsSplitTest` | 8 | 0 | 0 | 0 |
| **Total** | **350** | **0** | **0** | **3** |
### Failsafe (integration tests)
Full impacted-IT surface = **94 concrete classes** (`forkCount=2`,
`reuseForks=false`, `-Djacoco.skip=true`), run across three sessions. All
EXPLAIN assertions pass.
| Phase | Classes | Tests run | Pass | Fail | Error (env.) | Skip |
|---|---:|---:|---:|---:|---:|---:|
| Surefire | 9 | 350 | 350 | 0 | 0 | 3 |
| Failsafe | 94 | 3,215 | 3,185 | 0 | 30 | 93 |
| **All** | **103** | **3,565** | **3,535** | **0** | **30** | **96** |
Zero EXPLAIN-assertion failures across the entire impacted surface.
--
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]
