On Thu, Feb 23, 2017 at 8:06 PM, Andrew Dunstan
<andrew.duns...@2ndquadrant.com> wrote:
> While reviewing the IF NOT EXISTS patch for CREATE USER MAPPING I
> noticed that in several places we treat the user name as the name of the
> user mapping. Strictly ISTM that user mappings are really anonymous
> objects, so instead of something like user "mapping \"%s\" does not
> exist for the server" we should possibly have "user mapping for user
> \"%s\" does not exist for the server".

Your proposed usage is better than the existing one.

> I was about to make that change
> in the patch when I saw that it was consistent with current usage. Do we
> want to stick with the current usage where we treat the user name as the
> mapping name, or change it?

We should change existing usage and then commit the patch with new
usage. The new message being added should be consistent with other

Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company

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

Reply via email to