Hi Trevor,
+ + parent_index_info = BuildIndexInfo(parent_index);
The above is not used anywhere else in the code and seems redundant.
+ + ereport(NOTICE, + (errmsg("Index \"%s\" cloned.", + RelationGetRelationName(parent_index))));
DefineIndex will give out a message anyways for unique/primary keys. The above seems additional to it. Regards, Nikhils -- EnterpriseDB http://www.enterprisedb.com