Changes between Version 2 and Version 3 of Ticket #34262, comment 8
- Timestamp:
- Aug 13, 2023, 1:59:49 AM (16 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #34262, comment 8
v2 v3 41 41 }}} 42 42 43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look awkward for me.43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" looks awkward. 44 44 With `.values_list("least", flat=True)` clause present, there was a obvious reason for "MIN(`aggregation_book`.`pages`) min_pages" to be pushed down because it is a dependency for `least`, but without `.values_list("least", flat=True)` it loses it's reason to be pushed down. 45 45 I am a bit suspicious that choosing which additional item to be pushed down by looking at `values_list` worth it's effort considering frequency of this use case is thought to be small.