apurtell commented on PR #2513: URL: https://github.com/apache/phoenix/pull/2513#issuecomment-4675958991
## Test Results | Layer | Suites | Passed | Errors | Skipped | Notes | |---|---|---|---|---|---| | Connectionless unit tests | 6 | 341 | 0 | 3 | — | | Integration tests (minicluster) | 33 | 1204 | 4 | 14 | 4 pre-existing, unrelated (see below) | | **Total** | **39** | **1545** | **4** | **17** | green except documented pre-existing flake | ### Unit tests | Suite | Tests | Failures | Errors | Skipped | |---|---|---|---|---| | `query.explain.ExplainPlanTest` | 63 | 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 | | **Subtotal** | **341** | **0** | **0** | **3** | ### Integration tests #### Index suites | Suite | Tests | Failures | Errors | Skipped | |---|---|---|---|---| | `end2end.index.GlobalIndexCheckerIT` | 112 | 0 | 0 | 6 | | `end2end.index.MutableIndexIT` | 90 | 0 | 0 | 0 | | `end2end.index.PartialIndexIT` | 60 | 0 | 0 | 0 | | `end2end.index.IndexUsageIT` | 39 | 0 | 0 | 0 | | `end2end.index.SingleCellIndexIT` | 14 | 0 | 0 | 0 | | `end2end.index.GlobalIndexOptimizationIT` | 7 | 0 | 0 | 0 | | `end2end.index.SaltedIndexIT` | 2 | 0 | 0 | 0 | | `end2end.index.LocalIndexIT` | 60 | 0 | 4 ⚠ | 0 | #### Join suites | Suite | Tests | Failures | Errors | Skipped | |---|---|---|---|---| | `end2end.join.HashJoinGlobalIndexIT` | 33 | 0 | 0 | 0 | | `end2end.join.HashJoinLocalIndexIT` | 34 | 0 | 0 | 0 | | `end2end.join.HashJoinNoIndexIT` | 33 | 0 | 0 | 0 | | `end2end.join.SortMergeJoinGlobalIndexIT` | 35 | 0 | 0 | 0 | | `end2end.join.SortMergeJoinLocalIndexIT` | 35 | 0 | 0 | 0 | | `end2end.join.SortMergeJoinNoIndexIT` | 35 | 0 | 0 | 0 | #### Cost / optimizer | Suite | Tests | Failures | Errors | Skipped | |---|---|---|---|---| | `end2end.CostBasedDecisionIT` | 20 | 0 | 0 | 0 | #### View / tenant suites | Suite | Tests | Failures | Errors | Skipped | |---|---|---|---|---| | `end2end.index.ViewIndexIT` | 32 | 0 | 0 | 2 | | `end2end.TenantSpecificViewIndexIT` (covers `BaseTenantSpecificViewIndexIT`) | 11 | 0 | 0 | 0 | | `end2end.TenantSpecificViewIndexSaltedIT` | 12 | 0 | 0 | 0 | | `end2end.index.ChildViewsUseParentViewIndexIT` | 2 | 0 | 0 | 0 | | `end2end.ReadOnlyViewOnReadOnlyIT` | 2 | 0 | 0 | 0 | #### Other EXPLAIN-asserting suites | Suite | Tests | Failures | Errors | Skipped | Notes | |---|---|---|---|---|---| | `end2end.InListIT` | 228 | 0 | 0 | 0 | point lookup, data table, only candidate | | `end2end.ExplainPlanWithStatsEnabledIT` | 28 | 0 | 0 | 0 | `NO_INDEX` hint paths; estimate parsing | | `end2end.DeleteIT` | 74 | 0 | 0 | 0 | — | | `end2end.SequenceIT` | 56 | 0 | 0 | 0 | — | | `end2end.Bson4IT` | 32 | 0 | 0 | 6 | range scan / point lookup attributes | | `end2end.join.SubqueryIT` | 36 | 0 | 0 | 0 | audited — data-table-only (not annotated) | | `end2end.join.SubqueryUsingSortMergeJoinIT` | 21 | 0 | 0 | 0 | audited — data-table-only (not annotated) | | `end2end.UnionAllIT` | 19 | 0 | 0 | 0 | — | | `end2end.CursorWithRowValueConstructorIT` | 18 | 0 | 0 | 0 | first-line scan assertion only | | `end2end.json.JsonFunctionsIT` | 15 | 0 | 0 | 0 | more bound PK columns, data table | | `end2end.QueryLoggerIT` | 7 | 0 | 0 | 0 | logged vs. fresh EXPLAIN (self-consistent) | | `end2end.ReverseScanIT` | 4 | 0 | 0 | 0 | more bound PK columns, data table | | `end2end.UpgradeNamespaceIT` | 2 | 0 | 0 | 0 | — | #### Pre-existing failures (not introduced by this change) `end2end.index.LocalIndexIT` reports 4 errors in `testLocalIndexReverseScanShouldReturnAllRows` and `testLocalIndexUsedForUncoveredOrderBy` (each ×`isNamespaceMapped`), all `StackOverflowError`. Tracked by PHOENIX-7893. -- 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]
