On Mon, Aug 27, 2012 at 05:44:32PM +0300, Marti Raudsepp wrote: > On Mon, Aug 27, 2012 at 4:50 PM, Bruce Momjian <br...@momjian.us> wrote: > > Where are we on this? > > TL;DR: Got a review, requires substantial work, current github branch > is slightly broken, will get back to this soon. > > Tom Lane sent a thorough review of the patch here: > http://archives.postgresql.org/pgsql-hackers/2012-03/msg00655.php > (very much appreciated!) > > I have addressed some smaller points from that list in my github > branch, but it still requires a substantial amount of work (in > particular, the bulk of this patch which is the recursion logic in > eval_const_expressions_mutator, needs to be changed to prevent > unnecessary CacheExpr insertions and to store intermediate state in > the context struct). > > I got a small fragment of this into PostgreSQL 9.2 as commit > 81a646febe87964725647a36d839f6b4b405f3ae. I rebased my github branch > on top of this commit, but the rebase introduced some test failures > that I have not tracked down yet. I don't know if it applies to git > HEAD any more. > > Sadly some other things intervened and I have not had the time to > return to hacking on this patch. But I am hopeful I can get it into > shape during the 9.3 cycle.
OK, thanks for the update, and your work on this. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
