From: | Manlio Perillo <manlio(dot)perillo(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PGSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: send Describe Portal message in PQsendPrepare |
Date: | 2013-02-12 12:00:57 |
Message-ID: | 511A2EF9.50109@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Il 11/02/2013 22:39, Tom Lane ha scritto:
> Manlio Perillo <manlio(dot)perillo(at)gmail(dot)com> writes:
>> What is the reason why PQsendPrepare function does not send a
>> Describe Portal message?
>
> That would add a round trip, no?
>
Well, no.
An extra round trip is required with current implementation, since I
need to call PQsendPrepare, wait for server response, call
PQsendDescribePrepared, wait for server response.
>> Sending a Describe Portal message, make it possible for PQsendPrepare
>> function to *return* a PGresult with more useful informations, instead
>> of just the result status.
>
> That's *definitely* wrong, because the entire point of the PQsend
> functions is they don't wait for a server response.
>
The PQsendQueryParams sends the following protocol messages:
* Parse
* Bind
* Describe Portal
* Execute
* Sync
and of course does not wait for each response, since this is done in the
state machine.
I noted that PQsendQueryParams sends a Describe Portal message, and I
found it strange that the same is not done by PQsendPrepare.
I wrote the patch to check if this was done due to some technical
reason, but this seems to not be the case.
> regards, tom lane
Regards Manlio Perillo
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iEYEARECAAYFAlEaLvkACgkQscQJ24LbaUSBgACgjASGXyTl+rpHWGAGk5nm7Fnj
T68Anin9iEfbLw75ObHJxU6yfIazEZDS
=ZmPu
-----END PGP SIGNATURE-----
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-02-12 12:13:24 | Re: Support for REINDEX CONCURRENTLY |
Previous Message | Andres Freund | 2013-02-12 11:47:13 | Re: Support for REINDEX CONCURRENTLY |