An OID return value was being used only for a (rather pointless) assert.
Silence by removing the variable and the assert.
Per note from Peter Geoghegan
bool nulls[Natts_pg_conversion];
Datum values[Natts_pg_conversion];
NameData cname;
- Oid oid;
ObjectAddress myself,
referenced;
tup = heap_form_tuple(tupDesc, values, nulls);
/* insert a new tuple */
- oid = simple_heap_insert(rel, tup);
- Assert(OidIsValid(oid));
+ simple_heap_insert(rel, tup);
/* update the index if any */
CatalogUpdateIndexes(rel, tup);