Make TupleDescInitBuiltinEntry throw error for unsupported types.

Previously, it would just pass back a partially-uninitialized tupdesc,
which doesn't seem like a safe or useful behavior.

Backpatch to v10 where this code came in.

Discussion: https://postgr.es/m/30830.1544384975@sss.pgh.pa.us

diff --git a/src/backend/access/common/tupdesc.c b/src/backend/access/common/tupdesc.c
index b0434b467208bc19e2a3bd76453a3a1cedc31235..852b96ebbbf6ebc2db4ef2848c3cce1c4ccb8c4a 100644 (file)
--- a/src/backend/access/common/tupdesc.c
+++ b/src/backend/access/common/tupdesc.c
@@ -748,6 +748,9 @@ TupleDescInitBuiltinEntry(TupleDesc desc,
            att->attstorage = 'p';
            att->attcollation = InvalidOid;
            break;
+
+       default:
+           elog(ERROR, "unsupported type %u", oidtypeid);
    }
 }