Re: missing rename support

Dean Rasheed <dean.a.rasheed@gmail.com>

View thread View on postgr.es

From: Dean Rasheed <dean.a.rasheed@gmail.com>

To: Ali Dar <ali.munir.dar@gmail.com>

Cc: pgsql-hackers@postgresql.org

Date: 2013-02-03T15:04:48Z

Lists: pgsql-hackers

Attachments

alter-rule-rename_complete.v2.patch (application/octet-stream) patch v2

On 29 January 2013 15:34, Ali Dar <ali.munir.dar@gmail.com> wrote:
> Please find attached the complete patch for alter rename rule. I have
> followed all the suggestions.

This looks good. I've tested it, and it appears to work as intended.
I'm happy with the code, and the new docs and regression tests look
OK.

I have a couple of minor tweaks (see attached):

* On the new manual page, I replaced "table" with "table or view".

* In the new tab-completion code, I modified the query so that it
completes with tables as well as views, and limited the results to
just those relations that have a rule with the name specified,
otherwise the list of completions could be very long.

If you're happy with these changes, I think this is ready for committer review.

Regards,
Dean