Re-allow user_catalog_table option for materialized views.

tglsfdc · tglsfdc · commit 6e00ba1e1705 · 2016-11-10T15:00:58.000-05:00
The reloptions stuff allows this option to be set on a matview. While it's questionable whether that is useful or was really intended, it does work, and we shouldn't change that in minor releases. Commit e3e66d8 disabled the option since I didn't realize that it was possible for it to be set on a matview. Tweak the test to re-allow it. Discussion: <19749.1478711862@sss.pgh.pa.us>
diff --git a/src/include/utils/rel.h b/src/include/utils/rel.h
@@ -253,8 +253,9 @@ typedef struct StdRdOptions
  *		from the pov of logical decoding.  Note multiple eval or argument!
  */
 #define RelationIsUsedAsCatalogTable(relation)	\
-	((relation)->rd_rel->relkind == RELKIND_RELATION && \
-	 (relation)->rd_options ? \
+	((relation)->rd_options && \
+	 ((relation)->rd_rel->relkind == RELKIND_RELATION || \
+	  (relation)->rd_rel->relkind == RELKIND_MATVIEW) ? \
 	 ((StdRdOptions *) (relation)->rd_options)->user_catalog_table : false)
 
 
