shrish purohit <shrishpuro...@gmail.com> wrote:

> Updated Kedar's patch sending on behalf of Kedar.
> There are not toomany modifications. It includes improvements for validating
> multi-column partitioned tables.

Thanks, but I'd like to keep the first patch as possible as simple.
I'll refer your work in the *next* step, especially partition
triggers and ALTER commands.

I glanced over your patch and find some codes that have room for improvement:

  * Need to avoid RANGE and PARTITION to be reserved keywords.
  * Non SQL-based operations are preferred if possible, so need to
    reconsider pg_parse_query().
  * Adding NOTICE message to completionTag is a very bad idea.
    we don't need such feature because users can create trigger
    on the overflow partition to send those messages.

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center



-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to