Re: Push down time-related SQLValue functions to foreign server - Mailing list pgsql-hackers

From Alexander Pyhalov
Subject Re: Push down time-related SQLValue functions to foreign server
Date
Msg-id 2641ccef46e433bf088edc3f8569be6b@postgrespro.ru
Whole thread Raw
In response to Re: Push down time-related SQLValue functions to foreign server  (Ranier Vilela <ranier.vf@gmail.com>)
Responses Re: Push down time-related SQLValue functions to foreign server
List pgsql-hackers
Ranier Vilela писал 2021-08-20 14:19:

> Another question:
> For 0002 patch:
> 
> + if (node->funcid == F_NOW)
> + {
> + SQLValueFunction *svf = makeNode(SQLValueFunction);
> +
> + svf->op = SVFOP_CURRENT_TIMESTAMP;
> + svf->type = TIMESTAMPTZOID;
> + svf->typmod = -1;
> + svf->location = -1;
> +
> + deparseSQLValueFunction(svf, context);
> +
> + return;
> + }
> It seems to me that the svf->xpr field ( SQLValueFunction *svf ) is
> not initialized somewhere even by deparseSQLValueFunction.
> If it's not really used, it should be initialized to NULL, ok?
> 

xpr field just carries node type, which will be initialized by 
makeNode().

-- 
Best regards,
Alexander Pyhalov,
Postgres Professional



pgsql-hackers by date:

Previous
From: Matthias van de Meent
Date:
Subject: Re: NAMEDATALEN increase because of non-latin languages
Next
From: Jesper Pedersen
Date:
Subject: Re: Middleware Messages for FE/BE