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


Ignore:
Timestamp:
Aug 12, 2023, 4:19:33 AM (15 months ago)
Author:
Jonny Park

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34262, comment 8

    initial v1  
    4141}}}
    4242
    43 I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look award for me.
     43I think the position of "MIN(`aggregation_book`.`pages`) min_pages" look awkward for me.
    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