github-actions[bot]
commented
2 months ago Conformance comparison report-Cross Engine
| Base (legacy) | eval | +/- | |
|---|---|---|---|
| % Passing | 92.51% | 90.70% | -1.80% |
| :white_check_mark: Passing | 5382 | 5278 | -104 |
| :x: Failing | 436 | 541 | 105 |
| :large_orange_diamond: Ignored | 0 | 0 | 0 |
| Total Tests | 5818 | 5819 | 1 |
Number passing in both: 5071
Number failing in both: 229
Number passing in legacy engine but fail in eval engine: 312
Number failing in legacy engine but pass in eval engine: 207 :interrobang: CONFORMANCE REPORT REGRESSION DETECTED :interrobang: The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact. 207 test(s) were failing in legacy but now pass in eval. Before merging, confirm they are intended to pass. The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact.
Conformance comparison report-Cross Commit-LEGACY
| Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) | 4219bb392a9b2b30eb1281349be286b8b653bfed | +/- | |
|---|---|---|---|
| % Passing | 92.51% | 92.51% | 0.00% |
| :white_check_mark: Passing | 5382 | 5382 | 0 |
| :x: Failing | 436 | 436 | 0 |
| :large_orange_diamond: Ignored | 0 | 0 | 0 |
| Total Tests | 5818 | 5818 | 0 |
Number passing in both: 5382
Number failing in both: 436
Number passing in Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) but now fail: 0
Number failing in Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) but now pass: 0
Conformance comparison report-Cross Commit-EVAL
| Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) | 4219bb392a9b2b30eb1281349be286b8b653bfed | +/- | |
|---|---|---|---|
| % Passing | 90.70% | 90.70% | 0.00% |
| :white_check_mark: Passing | 5278 | 5278 | 0 |
| :x: Failing | 541 | 541 | 0 |
| :large_orange_diamond: Ignored | 0 | 0 | 0 |
| Total Tests | 5819 | 5819 | 0 |
Number passing in both: 5278
Number failing in both: 541
Number passing in Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) but now fail: 1
Number failing in Base (23f6feeb336d933d563f51e99b9a7d751bb3a9eb) but now pass: 1 :interrobang: CONFORMANCE REPORT REGRESSION DETECTED :interrobang:. The following test(s) were previously passing but now fail:
Click here to see
- Example 6 — Value Coercion, compileOption: LEGACYThe following test(s) were previously failing but now pass. Before merging, confirm they are intended to pass:
Relevant Issues
Description
internalnamespaceinternalnamespacepartiql-parserREADME that the ANTLR-generated sources are not to be used outside of code within this package.Other Information
Updated Unreleased Section in CHANGELOG: [NO]
Any backward-incompatible changes? [YES]
publicANTLR-generated sources under theinternalnamespace. We did not intend for these be used by external consumers ofpartiql-parser.Any new external dependencies? [NO]
Do your changes comply with the Contributing Guidelines and Code Style Guidelines? [YES]
License Information
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.