| From: | Marti Raudsepp <marti(at)juffo(dot)org> |
|---|---|
| To: | Jaime Casanova <jaime(at)2ndquadrant(dot)com>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Josh Berkus <josh(at)agliodbs(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Re: [PATCH] Caching for stable expressions with constant arguments v3 |
| Date: | 2011-12-05 23:05:42 |
| Message-ID: | CABRT9RD224HSRRNn8YTW8Po5O23vYfPptUZVwBUT7BzT9z-72A@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, Dec 4, 2011 at 22:53, Heikki Linnakangas
<heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
> This seems to have bitrotted, thanks to the recent refactoring in
> eval_const_expressions().
So here's the rebased version of this. It probably took me longer to
rebase this than it took Andres to write his patch ;)
Passes all regression tests and diffing clauses.c seems to have
introduced no functional differences.
But the good news is that I learned how to use git rebase and
pgindent, so it was worth it.
I will look into addressing some of the comments tomorrow, much thanks
for the feedback everyone!
Regards,
Marti
| Attachment | Content-Type | Size |
|---|---|---|
| cacheexpr-v3-rebased.patch | text/x-patch | 84.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tomas Vondra | 2011-12-05 23:25:36 | Re: why local_preload_libraries does require a separate directory ? |
| Previous Message | Bruce Momjian | 2011-12-05 22:06:37 | pg_upgrade and relkind filtering |