forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit650663b
committed
Use appropriate tuple descriptor in FDW batching
The FDW batching code was using the same tuple descriptor both for allslots (regular and plan slots), but that's incorrect - the subplan mayuse a different descriptor. Currently this is benign, because batchingis used only for INSERTs, and in that case the descriptors always match.But that would change if we allow batching UPDATEs.Fix by copying the appropriate tuple descriptor. Backpatch to 14, wherethe FDW batching was implemented.Author: Amit LangoteBackpatch-through: 14, where FDW batching was addedDiscussion:https://postgr.es/m/CA%2BHiwqEWd5B0-e-RvixGGUrNvGkjH2s4m95%3DJcwUnyV%3Df0rAKQ%40mail.gmail.com1 parentba95829 commit650663b
1 file changed
+4
-2
lines changedLines changed: 4 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
709 | 709 |
| |
710 | 710 |
| |
711 | 711 |
| |
712 |
| - | |
| 712 | + | |
713 | 713 |
| |
714 | 714 |
| |
715 | 715 |
| |
716 | 716 |
| |
| 717 | + | |
| 718 | + | |
717 | 719 |
| |
718 | 720 |
| |
719 | 721 |
| |
720 | 722 |
| |
721 | 723 |
| |
722 |
| - | |
| 724 | + | |
723 | 725 |
| |
724 | 726 |
| |
725 | 727 |
| |
|
0 commit comments
Comments
(0)