From: Mason S. <ms...@tr...> - 2013-11-05 18:35:37
|
On Mon, Nov 4, 2013 at 8:17 PM, Koichi Suzuki <koi...@gm...> wrote: > Mason; > > Thanks for this big patch. I understand the intention and would like > other members to look at this patch too. > Sure. > > I'd like to know your idea if this can be a part of the TRIGGER > improvement? > What did you have in mind? Yes, this impacts triggers as well. Regards, Mason > > > 2013/11/2 Mason Sharp <ms...@tr...> > >> Please see attached patch that tries to address the issue of XC using >> CTID for replicated updates and deletes when it is evaluated at a >> coordinator instead of being pushed down. >> >> The problem here is that CTID could be referring to a different tuple >> altogether on a different data node, which is what happened for one of our >> Postgres-XC support customers, leading to data issues. >> >> Instead, the patch looks for a primary key or unique index (with the >> primary key preferred) and uses those values instead of CTID. >> >> The patch could be improved further. Extra parameters are set even if >> not used in the execution of the prepared statement sent down to the data >> nodes. >> >> >> -- Mason Sharp TransLattice - http://www.translattice.com Distributed and Clustered Database Solutions |