On 08.09.21 04:04, Kyotaro Horiguchi wrote:
At Tue, 7 Sep 2021 11:22:24 +0200, Peter Eisentraut
wrote in
On 30.08.21 04:13, Kyotaro Horiguchi wrote:
+ else if (IsA(obj, Integer))
+ _outInteger(str, (Integer *) obj);
+ else if (IsA(obj, Float))
+ _outFlo
At Tue, 7 Sep 2021 11:22:24 +0200, Peter Eisentraut
wrote in
> On 30.08.21 04:13, Kyotaro Horiguchi wrote:
> > + else if (IsA(obj, Integer))
> > + _outInteger(str, (Integer *) obj);
> > + else if (IsA(obj, Float))
> > + _outFloat(str, (Float *) obj);
> > I felt that the t
+ char *type = strVal(lfirst(cell));
if (strcmp(type, "ndistinct") == 0)
{
--
2.33.0
From 6eb5ddb6b46446731df38ac5288df2feb17efd7c Mon Sep 17 00:00:00 2001
From: Peter Eisentraut
Date: Tue, 7 Sep 2021 11:11:28 +0200
Subject: [PATCH v2 2
Agree to the motive and +1 for the concept.
At Wed, 25 Aug 2021 15:00:13 +0100, Dagfinn Ilmari Mannsåker
wrote in
> However, the patch adds:
>
> > +typedef struct Null
> > +{
> > + NodeTag type;
> > + char *val;
> > +} Null;
>
> which doesn't seem to be used anywhere. Is tha
On Wed, Aug 25, 2021 at 9:49 PM Robert Haas wrote:
>
> On Wed, Aug 25, 2021 at 9:20 AM Peter Eisentraut
> wrote:
> > This change removes the Value struct and node type and replaces them
> > by separate Integer, Float, String, and BitString node types that are
> > proper node types and structs of
Peter Eisentraut writes:
> While trying to refactor the node support in various ways, the Value
> node is always annoying.
[…]
> This change removes the Value struct and node type and replaces them
> by separate Integer, Float, String, and BitString node types that are
> proper node types and str
On Wed, Aug 25, 2021 at 9:20 AM Peter Eisentraut
wrote:
> This change removes the Value struct and node type and replaces them
> by separate Integer, Float, String, and BitString node types that are
> proper node types and structs of their own and behave mostly like
> normal node types.
+1. I not
ckend/commands/statscmds.c
@@ -310,7 +310,7 @@ CreateStatistics(CreateStatsStmt *stmt)
build_mcv = false;
foreach(cell, stmt->stat_types)
{
- char *type = strVal((Value *) lfirst(cell));
+ char *type = strVal(lfirst(cell));
