apurtell commented on PR #2522: URL: https://github.com/apache/phoenix/pull/2522#issuecomment-4686577355
## Test Results | Suite | Tests run | Failures | Errors | Skipped | Result | |---|---|---|---|---|---| | Unit tests (full `phoenix-core` suite) | 2539 | 0 | 0 | 9 | PASS | | Integration tests (impacted subset) | 69 | 0 | 0 | 6 | PASS | ### Unit tests Result: `Tests run: 2539, Failures: 0, Errors: 0, Skipped: 9` — `BUILD SUCCESS`. The count rose from the previous baseline of 2537 to 2539 because two new connectionless cases were added to `ExplainPlanTest`: - `testJsonFunctionProjection` — validates `SERVER JSON PROJECTION 1` plus the `JSON_VALUE(JSONCOL, '$.type')` detail line. - `testBsonValueProjection` — validates `SERVER BSON PROJECTION 1` plus the `BSON_VALUE(PAYLOAD, 'user.id', 'VARCHAR', )` detail line. The pre-existing `testArrayElementProjection` covers `SERVER ARRAY PROJECTION 1` plus `ARRAY_ELEM(A_STRING_ARRAY, 1)`. Directly relevant unit classes: | Class | Tests run | Result | |---|---|---| | `org.apache.phoenix.query.explain.ExplainPlanTest` | 85 | PASS | | `org.apache.phoenix.compile.QueryCompilerTest` | (full class) | PASS | ### Integration tests Result: `BUILD SUCCESS`. Per-class breakdown: | Class | Tests run | Failures | Errors | Skipped | Result | |---|---|---|---|---|---| | `org.apache.phoenix.end2end.json.JsonFunctionsIT` | 15 | 0 | 0 | 0 | PASS | | `org.apache.phoenix.end2end.ProjectArrayElemAfterHashJoinIT` | 2 | 0 | 0 | 0 | PASS | | `org.apache.phoenix.end2end.Bson2IT` | 2 | 0 | 0 | 0 | PASS | | `org.apache.phoenix.end2end.Bson3IT` | 16 | 0 | 0 | 0 | PASS | | `org.apache.phoenix.end2end.Bson4IT` | 32 | 0 | 0 | 6 | PASS | | `org.apache.phoenix.end2end.Bson5IT` | 2 | 0 | 0 | 0 | PASS | | **Total** | **69** | **0** | **0** | **6** | **PASS** | The behavior change touches three EXPLAIN code paths: - `SERVER ARRAY ELEMENT PROJECTION` → `SERVER ARRAY PROJECTION <n>` + detail lines. - `SERVER JSON FUNCTION PROJECTION` → `SERVER JSON PROJECTION <n>` + detail lines. - BSON path projections now emit a new `SERVER BSON PROJECTION <n>` clause. `JsonFunctionsIT` and `ProjectArrayElemAfterHashJoinIT` directly assert on the renamed JSON/array projection output and were migrated to the new `serverParsedProjections` API. The four `Bson*IT` suites assert table/iterator/scan-type attributes. Other explain-asserting ITs (e.g. `ExplainPlanWithStatsEnabledIT`) do not use array/JSON/BSON projections and were excluded. -- 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]
