Re: send Describe Portal message in PQsendPrepare

Поиск
Список
Период
Сортировка
От Manlio Perillo
Тема Re: send Describe Portal message in PQsendPrepare
Дата
Msg-id 511A2EF9.50109@gmail.com
обсуждение исходный текст
Ответ на Re: send Describe Portal message in PQsendPrepare  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Il 11/02/2013 22:39, Tom Lane ha scritto:
> Manlio Perillo <manlio.perillo@gmail.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-----



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Support for REINDEX CONCURRENTLY
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Support for REINDEX CONCURRENTLY