Changes between Initial Version and Version 1 of Ticket #34262, comment 8
- Timestamp:
- Aug 12, 2023, 4:19:33 AM (16 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #34262, comment 8
initial v1 41 41 }}} 42 42 43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look aw ard for me.43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look awkward for me. 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.