Re: foreign key on virtual generated column - Mailing list pgsql-hackers

From Srinath Reddy Sadipiralla
Subject Re: foreign key on virtual generated column
Date
Msg-id CAFC+b6pnURqakmx=2VJ+suz3KwwC9tdQ7AWZB4wJirV2G9J8dg@mail.gmail.com
Whole thread Raw
In response to Re: foreign key on virtual generated column  (Srinath Reddy Sadipiralla <srinath2133@gmail.com>)
List pgsql-hackers
Hi Jian,

did initial testing and review ,the patches LGTM except
indentation we can solve this by running pg_indent.

i have one doubt that why we are using compute_virtual
argument todo the same work in both cases, is it because
of future proofing ,like we may do something different for
stored and virtual in future?

/* No luck, so prepare the expression for execution */
if (attgenerated == ATTRIBUTE_GENERATED_STORED)
{
ri_GeneratedExprs[i] = ExecPrepareExpr(expr, estate);
ri_NumGeneratedNeeded++;
}
else if (compute_virtual)
{
ri_GeneratedExprs[i] = ExecPrepareExpr(expr, estate);
ri_NumGeneratedNeeded++;
}


--
Thanks,
Srinath Reddy Sadipiralla
EDB: https://www.enterprisedb.com/

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCH] Fix ALTER SYSTEM empty string bug for GUC_LIST_QUOTE parameters
Next
From: Andreas Karlsson
Date:
Subject: Re: [PATCH] Fix ALTER SYSTEM empty string bug for GUC_LIST_QUOTE parameters