Greetings, I've reviewed the patch here: http://archives.postgresql.org/message-id/[EMAIL PROTECTED]
and am happy to report that it looks to be in good order. It has
documentation and regression updates, is in context diff format,
patches against current CVS with only some minor fuzz, and appears to
work as advertised. I looked over the patch and could easily follow
what was going on, did some additional testing beyond the regression
tests, and reviewed the documentation changes.
My only comment on this patch is that the documentation updates might
include a link back to Section 53.2 for more information, and/or to
the SET STORAGE portion of the alter table/alter column command
documentation, since the only other 'storage' reference in create
table's documentation is about table storage parameters.
Thanks!
Stephen
signature.asc
Description: Digital signature
