[Patch] change the return value of PQsendFlushRequest

Поиск
Список
Период
Сортировка
От zhangjie2@fujitsu.com
Тема [Patch] change the return value of PQsendFlushRequest
Дата
Msg-id TYWPR01MB76781872C2B69EC9CDB2FB04F91C9@TYWPR01MB7678.jpnprd01.prod.outlook.com
обсуждение исходный текст
Список pgsql-hackers
Hi all

The return value of function PQsendFlushRequest is 1 or 0.
-------------------------------------------------------------------
<para>
    Sends a request for the server to flush its output buffer.
<synopsis>
int PQsendFlushRequest(PGconn *conn);
</synopsis>
</para>

<para>
    Returns 1 for success.  Returns 0 on any failure.
</para>
---------------------------------------------------------------------
But in the following code, false is returned.
I think it would be better to change to 0.

int PQsendFlushRequest(PGconn *conn)
{
......
    if (conn->asyncStatus != PGASYNC_IDLE &&
        conn->pipelineStatus == PQ_PIPELINE_OFF)
    {
        appendPQExpBufferStr(&conn->errorMessage,
                             libpq_gettext("another command is already in progress\n"));
        return false;  ※
    }
......
}

Best Regards!
Zhangjie


Вложения

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

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: Disable WAL logging to speed up data loading
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: Can a child process detect postmaster death when in pg_usleep?