alamb edited a comment on pull request #844:
URL: https://github.com/apache/arrow-rs/pull/844#issuecomment-950788091
> @Dandandan @alamb i wonder if you think defining lt et. al. makes sense
here, if we allow false < true. if that's defined then i can simplify this
branch
I personally think it does make sense to define the other logical comparison
operators for `BooleanArray` (even though the usecase for `bool` < `bool` is
limited). My primary rationale is for completeness (SQL and Postgres allows for
it) and I do think it will avoid having to special case boolean columns for
features like DataFusion partition pruning (which converts `=` into `<=` and
`>` comparisons for comparing to statistics)
Also, postgres supports it, FWIW
```
alamb=# select true < false;
?column?
----------
f
(1 row)
alamb=# select true > false;
?column?
----------
t
(1 row)
```
--
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]
