Print more information about getObjectIdentityParts() failures.

tglsfdc · tglsfdc · commit a486841eb115 · 2014-12-31T14:44:43.000-05:00
This might help us debug what's happening on some buildfarm members.

In passing, reduce the message from ereport to elog --- it doesn't seem
like this should be a user-facing case, so not worth translating.
diff --git a/src/backend/catalog/objectaddress.c b/src/backend/catalog/objectaddress.c
@@ -4035,13 +4035,12 @@ getObjectIdentityParts(const ObjectAddress *object,
 
 	/*
 	 * If a get_object_address representation was requested, make sure we are
-	 * providing one.  We don't check for objargs, because many of the cases
-	 * above leave it as NIL.
+	 * providing one.  We don't check objargs, because many of the cases above
+	 * leave it as NIL.
 	 */
 	if (objname && *objname == NIL)
-		ereport(ERROR,
-				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
-				 errmsg("requested object address for object type that cannot support it")));
+		elog(ERROR, "requested object address for unsupported object class %d: text result \"%s\"",
+			 (int) getObjectClass(object), buffer.data);
 
 	return buffer.data;
 }
