public inbox for [email protected]  
help / color / mirror / Atom feed
From: Peter Eisentraut <[email protected]>
To: Shlok Kyal <[email protected]>
Cc: pgsql-hackers <[email protected]>
Cc: jian he <[email protected]>
Cc: Dean Rasheed <[email protected]>
Subject: Re: Virtual generated columns
Date: Tue, 4 Feb 2025 23:36:53 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <CANhcyEVEgQLjJnca9R75ptHE+wVWrVoy8iVyO6_93p+q_BKZQA@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<CAEZATCVLfMzNwT1J0bnvtv6YbmNSN9QSXb+1QSGyX6vK4q67Ow@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<CAEZATCUHnmOu7RtDuP+PCtS_dqvU8hFsAYYFPdiB8gyFszj5WQ@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<CAEZATCXQcjZPq+OsczFc13M6v+OP_NiqmUAcRAzi1K60KFuWmQ@mail.gmail.com>
	<[email protected]>
	<CAEZATCXnp5XTkUCsmfjLXVWZ7TaibWatnXDdu52J6zN3XsyZhw@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<CANhcyEVEgQLjJnca9R75ptHE+wVWrVoy8iVyO6_93p+q_BKZQA@mail.gmail.com>

On 28.01.25 10:40, Shlok Kyal wrote:
> Test 5: Update publication on non virtual gen with no column list specified
> 
> CREATE TABLE t1 (a int, b int GENERATED ALWAYS AS (a * 2) VIRTUAL);
> create publication pub1 for table t1;
> alter table t1 replica identity full;
> update t1 set a = 10;
> 
> No error is thrown, and an update is happening. It should have thrown
> an ERROR as the unpublished generated column 'b' is part of the
> replica identity.

Thanks, I have fixed that in v14.  (The other 4 tests were correct, right?)






view thread (75+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Virtual generated columns
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox