Changes between Version 2 and Version 3 of Ticket #34262, comment 8


Ignore:
Timestamp:
Aug 13, 2023, 1:59:49 AM (9 months ago)
Author:
Jonny Park

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34262, comment 8

    v2 v3  
    4141}}}
    4242
    43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look awkward for me.
     43I think the position of "MIN(`aggregation_book`.`pages`) min_pages" looks awkward.
    4444With `.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.
    4545I 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.
Back to Top