Re: BUG #14314: Mismatch of comment of a function.

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Cc: t(dot)katsumata1122(at)gmail(dot)com, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #14314: Mismatch of comment of a function.
Date: 2016-09-22 21:34:29
Message-ID: 20160922213429.GA18637@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs


Thanks, patch applied.

---------------------------------------------------------------------------

On Mon, Sep 12, 2016 at 11:04:14AM +1000, Haribabu Kommi wrote:
>
>
> On Wed, Sep 7, 2016 at 11:42 PM, <t(dot)katsumata1122(at)gmail(dot)com> wrote:
>
> The following bug has been logged on the website:
>
> Bug reference:      14314
> Logged by:          Tomonari Katsumata
> Email address:      t(dot)katsumata1122(at)gmail(dot)com
> PostgreSQL version: 9.6rc1
> Operating system:   Source Code
> Description:
>
> Hello.
>
> I noticed that it isn't proper for a comment  for
> transformOnConflictClause.
>
> ---
>  940 /*
>  941  * transformSelectStmt -
>  942  *    transforms an OnConflictClause in an INSERT
>  943  */
>  944 static OnConflictExpr *
>  945 transformOnConflictClause(ParseState *pstate,
>  946                           OnConflictClause *onConflictClause)
>  947 {
> ----
>
> It should be "transformOnConflictClause" rather than
> "transformSelectStmt".
>
>
> Attached patch fixes the reported problem. The patch needs to be
> applied till the 9.5 stable branch where the on conflict is introduced.
>
>
> Regards,
> Hari Babu
> Fujitsu Australia

> diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c
> index eac86cc..d9085fe 100644
> --- a/src/backend/parser/analyze.c
> +++ b/src/backend/parser/analyze.c
> @@ -938,7 +938,7 @@ transformInsertRow(ParseState *pstate, List *exprlist,
> }
>
> /*
> - * transformSelectStmt -
> + * transformOnConflictClause -
> * transforms an OnConflictClause in an INSERT
> */
> static OnConflictExpr *

>
> --
> Sent via pgsql-bugs mailing list (pgsql-bugs(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-bugs

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message alain_peters 2016-09-23 08:00:01 BUG #14336: Using rls on a table, suddenly the table is missing
Previous Message Robert Haas 2016-09-22 20:57:02 Re: Want a routine vacuum for postgres