Fix logical eval ops w/ MISSING arg

[alancai98]

#456

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.70%. Comparing base (17886f3) to head (e5a1838).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #457      +/-   ##
==========================================
+ Coverage   79.38%   80.70%   +1.32%     
==========================================
  Files          67       66       -1     
  Lines       17912    17638     -274     
  Branches    17912    17638     -274     
==========================================
+ Hits        14219    14235      +16     
+ Misses       3118     2964     -154     
+ Partials      575      439     -136     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Conformance comparison report

	Base (17886f3)	84ce8cf	+/-
% Passing	90.35%	90.34%	-0.02%
✅ Passing	5731	5730	-1
❌ Failing	612	613	1
🔶 Ignored	0	0	0
Total Tests	6343	6343	0

Number passing in both: 5730

Number failing in both: 612

Number passing in Base (17886f3) but now fail: 1

Number failing in Base (17886f3) but now pass: 0

⁉️ CONFORMANCE REPORT REGRESSION DETECTED ⁉️. The following test(s) were previously passing but now fail:

Click here to see

• partiql_tests::eval::spec_tests::section_7::permissive_data_type_mismatch_in_logical_expression

[alancai98]

TODO: figure out the permissive behavior of logical operands with mistyped arguments. Should they return MISSING or NULL:

• NOT { 'a': 1 }
• TRUE AND {'a': 1}