View Single Post

   
  #3 (permalink)  
Old 04-18-2008, 01:16 AM
Tom Lane
 
Posts: n/a
Default Re: Libpq COPY optimization patch

"Alon Goldshuv" <agoldshuv@greenplum.com> writes:
> I guess that although parseInput is cheap we could still use a conditional
> to see when data was sent and only then call it (without PQconsumeInput)
> instead of calling it every single time PQputCopyData is called. Any
> objection to that?


You mean something like

if (input-buffer-not-empty)
parseInput();

? This still bothers me a bit since it's a mixing of logic levels;
PQputCopyData is an output routine, it shouldn't be getting its fingers
dirty with input buffer contents. I'm willing to tolerate this if it
can be demonstrated that it provides a useful performance gain compared
to the unconditional parseInput call, but let's see some numbers.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply With Quote