The patch I reviewd is listed among CommitFest 2012-09 patches. I hope it can be of some help, despite the fact, that I don't really feel like being terribly qualified here.
Submission review ---- The patch goes cleanly against v9.3, as fetched by clonig current HEAD from postgres git repository: $ git clone .. $ ./configure --with-readline $ make $ make world $ make check Apart from problems with substituted readline linkage, all that is this patch related went smoothly. The patch contains signifficant amount of testing code, so unit test it provides gives confidence, that the patch works as intended. Usability review ---- > Do we want that? > Do we already have it? As long as I'm concerned, this patch solves dataset partitioning problem, which is hard to cope with otherwise. The nested storage this patch provides solves the problem for me. > Does the patch actually implement that? Current version of the patch does not look like implementing "ON UPDATE CASCADE" and "ON UPDATE DELETE", just yet; but this is documented.... Apart from that, basic FK functionality work fine. Feature test ---- Before running "make check", I've made a few tests off the top of my head, and everything was correct and intuitive. "make check" also went flowlessly. So the patch looks fine for me. -R -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
