Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> Here's a patch.
Looks reasonable, but please try a little harder on the comments for the
new function --- IMO it should have a header comment that explains what
it's supposed to do exactly.
> For some reason, AlterSeqNamespaces was being passed a schema name.
> This wasn't used and was not possible to keep in the patched code so I
> just removed it.
Probably a hangover from some previous state of the code. If it's not
used I see no reason not to remove it.
regards, tom lane
