Skip to content

Commit

Permalink
Remove Tuplesortstate.copytup function
Browse files Browse the repository at this point in the history
It's currently unclear how do we split functionality between
Tuplesortstate.copytup() function and tuplesort_put*() functions.
For instance, copytup_index() and copytup_datum() return error while
tuplesort_putindextuplevalues() and tuplesort_putdatum() do their work.
This commit removes Tuplesortstate.copytup() altogether, putting the
corresponding code into tuplesort_put*().

Discussion: https://postgr.es/m/CAPpHfdvjix0Ahx-H3Jp1M2R%2B_74P-zKnGGygx4OWr%3DbUQ8BNdw%40mail.gmail.com
Author: Alexander Korotkov
Reviewed-by: Pavel Borisov, Maxim Orlov, Matthias van de Meent
Reviewed-by: Andres Freund, John Naylor
  • Loading branch information
akorotkov committed Jul 27, 2022
1 parent ffd1b6b commit d47da31
Showing 1 changed file with 133 additions and 198 deletions.

0 comments on commit d47da31

Please sign in to comment.