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


Ignore:
Timestamp:
Aug 12, 2023, 4:20:20 AM (16 months ago)
Author:
Jonny Park

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34262, comment 8

    v1 v2  
    4242
    4343I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look awkward for me.
    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.
     44With `.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