forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitdc084d7
committed
Fix incorrect memory context switch in COPY TO execution
c532d15 has split the logic of COPY commands into multiple files, onechange being to move the internals of BeginCopy() to BeginCopyTo().Originally the code was written so as we'd switch back-and-forth betweenthe current execution memory context and the dedicated memory contextfor the COPY command, and this refactoring has introduced an extraswitch to the current memory context from the COPY context onceBeginCopyTo() is done with the past logic coming from BeginCopy().The code was correctly doing the analyze, rewrite and planning phases inthe COPY context, but it was not assigning "copy_file" (FILE* used whencopying to a source file) and "filename" in the COPY context, making theCOPY status data inconsistent.Author: Bharath RupireddyReviewed-by: Japin LiDiscussion:https://postgr.es/m/CALj2ACWvVa69foi9jhHFY=2BuHxAoYboyE+vXQTARwxZcJnVrQ@mail.gmail.comBackpatch-through: 141 parent0d72d6b commitdc084d7
1 file changed
+0
-2
lines changedLines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
657 | 657 | | |
658 | 658 | | |
659 | 659 | | |
660 | | - | |
661 | | - | |
662 | 660 | | |
663 | 661 | | |
664 | 662 | | |
| |||
0 commit comments
Comments
(0)