Add two missing cases to ATWrongRelkindError.

This way, we produce a better error message if someone tries to do
something like ALTER INDEX .. ALTER COLUMN .. SET STORAGE.

Amit Langote

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 403582ccd53bd9a88ef729f1bcdf94d88efd7271..a5bc508e15d6624c297679e5c02dbecfac3d6f46 100644 (file)
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -4336,6 +4336,9 @@ ATWrongRelkindError(Relation rel, int allowed_targets)
        case ATT_TABLE | ATT_VIEW:
            msg = _("\"%s\" is not a table or view");
            break;
+       case ATT_TABLE | ATT_VIEW | ATT_FOREIGN_TABLE:
+           msg = _("\"%s\" is not a table, view or foreign table");
+           break;
        case ATT_TABLE | ATT_VIEW | ATT_MATVIEW | ATT_INDEX:
            msg = _("\"%s\" is not a table, view, materialized view, or index");
            break;
@@ -4345,6 +4348,9 @@ ATWrongRelkindError(Relation rel, int allowed_targets)
        case ATT_TABLE | ATT_MATVIEW | ATT_INDEX:
            msg = _("\"%s\" is not a table, materialized view, or index");
            break;
+       case ATT_TABLE | ATT_MATVIEW | ATT_FOREIGN_TABLE:
+           msg = _("\"%s\" is not a table, materialized view, or foreign table");
+           break;
        case ATT_TABLE | ATT_FOREIGN_TABLE:
            msg = _("\"%s\" is not a table or foreign table");
            break;