Postgres Pro
Downloads
Home   >   mailing lists

Re: [HACKERS] psql - add special variable to reflect the last query status - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: [HACKERS] psql - add special variable to reflect the last query status
Date
Msg-id CAFj8pRCz9r2DjjD84Ga_uaoMJYoQoW+-rsuH0pfertpnN9LnGg@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] psql - add special variable to reflect the last querystatus  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: [HACKERS] psql - add special variable to reflect the last query status  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
Tree view


2017-06-17 7:58 GMT+02:00 Fabien COELHO <coelho@cri.ensmp.fr>:

I have not any other comments. The implementation is trivial. I rerun all
tests and tests passed.

I'll mark this patch as ready for commiters.

 Oops, I just noticed a stupid confusion on my part which got through, I was setting "ERROR" as "success", inverting the expected boolean value.

Here is a fixed version.

I missed it too.

We can introduce macro SetVariableBool(vars, varname, bool) instead

 SetVariable(pset.vars, "ERROR", "FALSE");

Regards

Pavel


--
Fabien.

pgsql-hackers by date:

Previous
From: Tatsuo Ishii
Date:
Subject: [HACKERS] REPLICA IDENTITY FULL
Next
From: Pavel Stehule
Date:
Subject: Re: [HACKERS] proposal psql \gdesc