Log In - Home Page

Patch: [PoC] Writable Foreign Tables

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-01
Topic Server Features
Patch Status Committed
Author KaiGai Kohei
Reviewers Alexander Korotkov, Laurenz Albe
Committer Tom Lane
Close Date 2013-03-10
Patch by kaigai on 2012-08-23 05:24:34 AM: Initial version.
Patch by kaigai on 2012-09-23 06:27:06 AM: revised patch, according to the idea of pseudo-column.
Patch by kaigai on 2012-09-24 08:50:56 AM: INTERNALOID is more suitable for "rowid" pseudo-column rather than VOIDOID
Comment by alvherre on 2012-11-05 06:17:24 PM: moved from CF2
Patch by kaigai on 2012-11-15 09:08:37 PM: Patch refreshed.
Review by laurenz on 2012-11-16 03:31:49 PM: Got an unexpected error for an update.
Unhappy with name and function of GetForeignRelInfo.
The patch should not change trigger code as much as it does.
Patch by kaigai on 2012-12-03 12:49:12 PM: PoC code based on Hanada-san's latest postgres_fdw v5, it also works fine on RDBMS backend.
Review by laurenz on 2012-12-08 11:13:55 AM: Encountered several crashes, one API design question.
Patch by kaigai on 2012-12-08 08:45:30 PM: v6 patch, the above crashes were eliminated
Patch by laurenz on 2012-12-11 03:58:45 PM: Try to improve the documentation. Encountered one query that deadlocks.
Patch by laurenz on 2012-12-14 02:59:14 PM: KaiGai addressed some problems and split the patch.
Patch by laurenz on 2012-12-14 03:00:09 PM: Small documentation fix.
Patch by kaigai on 2012-12-23 11:32:50 AM: Small changes of APIs to support remote DEFAULT and BEFORE-ROW triggers and consistent results in RETURNING.
Comment by ringerc on 2013-01-18 02:59:21 AM: Asked for committer
Comment by ringerc on 2013-01-18 03:13:55 AM: Tom intends to evaluate in next couple of days; check back 2013-01-24
Patch by kaigai on 2013-01-29 09:22:21 AM: v11, rebased to the latest master.
Patch by kaigai on 2013-02-07 05:05:04 PM: v12, add optimization on make_modifytable invocation by Daniel Farina
Comment by ringerc on 2013-03-03 02:17:14 PM: Queried status, partly to raise interest on-list.
Comment by ringerc on 2013-03-04 02:48:57 AM: Tom would like to see this commited but thinks it needs a lot of work first and hasn't had time to address it.

Add Comment

Please log in to comment on this patch.