The 8.2 patch that added support for an alias on the target table of

UPDATE/DELETE forgot to teach ruleutils.c to display the alias.
Per bug #4141 from Mathias Seiler.

diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c
index 7ce10562bfb61135864a835a7e7433e5db87c0de..8bd859c18ef2bc11bdc924983b9192f20a3b753c 100644 (file)
--- a/src/backend/utils/adt/ruleutils.c
+++ b/src/backend/utils/adt/ruleutils.c
@@ -2437,9 +2437,13 @@ get_update_query_def(Query *query, deparse_context *context)
                appendStringInfoChar(buf, ' ');
                context->indentLevel += PRETTYINDENT_STD;
        }
-       appendStringInfo(buf, "UPDATE %s%s SET ",
+       appendStringInfo(buf, "UPDATE %s%s",
                                         only_marker(rte),
                                         generate_relation_name(rte->relid));
+       if (rte->alias != NULL)
+               appendStringInfo(buf, " %s",
+                                                quote_identifier(rte->alias->aliasname));
+       appendStringInfoString(buf, " SET ");
 
        /* Add the comma separated list of 'attname = value' */
        sep = "";
@@ -2511,12 +2515,15 @@ get_delete_query_def(Query *query, deparse_context *context)
        Assert(rte->rtekind == RTE_RELATION);
        if (PRETTY_INDENT(context))
        {
-               context->indentLevel += PRETTYINDENT_STD;
                appendStringInfoChar(buf, ' ');
+               context->indentLevel += PRETTYINDENT_STD;
        }
        appendStringInfo(buf, "DELETE FROM %s%s",
                                         only_marker(rte),
                                         generate_relation_name(rte->relid));
+       if (rte->alias != NULL)
+               appendStringInfo(buf, " %s",
+                                                quote_identifier(rte->alias->aliasname));
 
        /* Add the USING clause if given */
        get_from_clause(query, " USING ", context);