partiql / partiql-tests

Test data for checking conformance to the PartiQL specification.
Apache License 2.0
7 stars 1 forks source link

Fix `GROUP BY` tests grouping over `MISSING` #77

Closed alancai98 closed 1 year ago

alancai98 commented 1 year ago

Addresses #73.

partiql-lang-kotlin and tests ported from partiql-lang-kotlin to partiql-tests had the incorrect behavior when a group key value was MISSING. MISSING should be coerced to NULL as per section 11.1.1 of the PartiQL spec. Previously, the group key would be MISSING in the edge case of when no other group keys values were NULL.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.