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.