Re: [Postgres-xc-developers] Fwd: [Postgres-xc-committers] Postgres-XC branch, REL1_2_STABLE, updated. XC1_0_BETA1_PG9_1-3999-ge5466f7

[Pavan D. <pav...@gm...>]

> On 10-Mar-2014, at 2:08 pm, Ashutosh Bapat <ash...@en...> wrote:
> 
> Here is another reason why we shouldn't add requirement for primary key in replicated tables. Thanks Abbas for pointing it out.
> 
> If the UPDATE/DELETE statement gets FQSed, we don't need a primary key for replicated table and if it doesn't, then we need a primary key. It will be difficult to explain users why some DMLs on replicated tables error out and why some of them don't.

We should  error out irrespective of whether the query can be FQSed or not.

> Again we come back to the backward compatibility.

Again, it's a bug

> Before anyone suggests, disabling FQS for replicated tables, is not a solution here. It will affect performance badly.

IMHO correctness takes precedence over performance, especially now that we are moving from being an experimental product to being production ready

FWIW I am ok with adding a non documented guc for now 

Thanks,
Pavan
>  
>>