Why in the world is reindex_relation defined like this?
#define REINDEX_CHECK_CONSTRAINTS 0x1
#define REINDEX_SUPPRESS_INDEX_USE 0x2
extern bool reindex_relation(Oid relid, bool toast_too, int flags);
Seems like a rational design would have folded toast_too in as another
flag bit, instead of keeping it a separate argument.
regards, tom lane