apurtell commented on PR #2525: URL: https://github.com/apache/phoenix/pull/2525#issuecomment-4696746046
## Test Results ### Unit tests (`*Test`) | Test class | Tests | Failures | Errors | Skipped | Result | | --- | --- | --- | --- | --- | --- | | `ExplainOptionsParserTest` (new) | 16 | 0 | 0 | 0 | PASS | | `QueryParserTest` | 93 | 0 | 0 | 0 | PASS | | `ExplainPlanTest` | 94 | 0 | 0 | 0 | PASS | | **Total** | **203** | **0** | **0** | **0** | **PASS** | `ExplainOptionsParserTest` covers the new grammar: `REGIONS`, `VERBOSE`, `FORMAT TEXT`, `FORMAT JSON`, case-insensitivity, option ordering, the legacy `WITH REGIONS` alias, and the error paths (unknown option, unknown format kind, duplicate option, `FORMAT` without a value, `REGIONS` with a value, and mixing the legacy alias with the option list). ### Integration tests (`*IT`) | Test class | Category | Tests | Failures | Errors | Skipped | Result | | --- | --- | --- | --- | --- | --- | --- | | `AggregateIT` | ParallelStatsDisabled | 25 | 0 | 0 | 0 | PASS | | `GlobalImmutableNonTxIndexIT` | ParallelStatsDisabled | 46 | 0 | 0 | 0 | PASS | | `LocalMutableNonTxIndexIT` | ParallelStatsDisabled | 46 | 0 | 0 | 4 | PASS | | `QueryLoggerIT` | NeedsOwnMiniCluster | 7 | 0 | 0 | 0 | PASS | | `FlappingLocalIndexIT` | NeedsOwnMiniCluster | 12 | 0 | 0 | 0 | PASS | - `AggregateIT` drives the three migrated sites in `BaseAggregateIT` (`testGroupByOrderPreserving`, `testSumGroupByOrderPreserving`, `testAvgGroupByOrderPreserving`). - `GlobalImmutableNonTxIndexIT` / `LocalMutableNonTxIndexIT` drive the three migrated sites in `BaseIndexIT` (`testIndexWithNullableFixedWithCols`, `testIndexWithNullableDateCol`, `testSelectAllAndAliasWithIndex`). The 4 skipped tests in `LocalMutableNonTxIndexIT` are pre-existing failing tests unrelated to this change. - `FlappingLocalIndexIT` was previously uncategorized and therefore skipped by all group-filtered failsafe executions. It is now annotated `@Category(NeedsOwnMiniClusterTest.class)`. -- 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]
