Changes between Version 1 and Version 2 of Ticket #33018, comment 2
- Timestamp:
- Aug 13, 2021, 11:44:22 AM (3 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #33018, comment 2
v1 v2 9 9 }}} 10 10 11 It also seems weird that we default to `0` as opposed to `NULL` which would be a more correct value .11 It also seems weird that we default to `0` as opposed to `NULL` which would be a more correct value for a non-coalesced annotation. 12 12 13 13 Alternatively we could adjust `Coalesce.as_sql` to catch `EmptyResultSet` when it's compiling its source expressions but that's more involved as most of the logic for that currently lives in `Func.as_sql`. We could also use both of these approaches.