| From: | Stephen Frost <sfrost(at)snowman(dot)net> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Ildar Musin <i(dot)musin(at)postgrespro(dot)ru>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: IndexTupleDSize macro seems redundant |
| Date: | 2018-01-12 02:17:19 |
| Message-ID: | 20180112021719.GU2416@tamriel.snowman.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Tom,
* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> Stephen Frost <sfrost(at)snowman(dot)net> writes:
> > Updated (combined) patch attached for review. I went through and looked
> > again to make sure there weren't any cases of making an unaligned
> > pointer to a struct and didn't see any, and I added some comments to
> > _bt_restore_page().
>
> Looks OK from here.
Great, thanks, I'll mark it as Ready For Committer then.
Robert, since you were on this thread and the patch is mostly yours
anyway, did you want to commit it? I'm happy to do so also, either way.
Thanks!
Stephen
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2018-01-12 02:19:26 | Re: Transform for pl/perl |
| Previous Message | Stephen Frost | 2018-01-12 02:14:14 | Re: Add default role 'pg_access_server_files' |