Remove references to reltoastidxid from gpcheckcat.
Remove references to reltoastidxid from gpcheckcat.

In the query in checkPGClass, AFAICS the reltoastrelid and reltoastidxid
fields were not used for anything, so just remove them from the query.
(To be honest, the whole query seems pretty pointless to me).

In the checkOwners() query, I also just removed the check for the toast
index. We have a regression test to check that the owner is set correctly
in ALTER TABLE commands, in the main test suite, in
'alter_distribution_policy' test, so it seems unlikely for that particular
problem to reappear. An index should always have the same owner as the
table itself, so perhaps adding a test for that more general case would be
useful, but I don't see much value in testing the toast index specifically.

pg_class.reltoastidxid field will be removed in PostgreSQL 9.4. Until
then, there's nothing wrong with these queries as such, but I wanted to
open a separate PR to get more eyeballs on the decisions.

jmcatamney/gpdb,xinzweb/gpdb,lisakowen/gpdb,greenplum-db/gpdb,adam8157/gpdb,50wu/gpdb,lisakowen/gpdb,adam8157/gpdb,ashwinstar/gpdb,greenplum-db/gpdb,greenplum-db/gpdb,lisakowen/gpdb,ashwinstar/gpdb,xinzweb/gpdb,jmcatamney/gpdb,50wu/gpdb,jmcatamney/gpdb,lisakowen/gpdb,lisakowen/gpdb,xinzweb/gpdb,50wu/gpdb,adam8157/gpdb,50wu/gpdb,lisakowen/gpdb,ashwinstar/gpdb,lisakowen/gpdb,ashwinstar/gpdb,adam8157/gpdb,ashwinstar/gpdb,adam8157/gpdb,50wu/gpdb,50wu/gpdb,jmcatamney/gpdb,jmcatamney/gpdb,xinzweb/gpdb,xinzweb/gpdb,ashwinstar/gpdb,greenplum-db/gpdb,xinzweb/gpdb,lisakowen/gpdb,ashwinstar/gpdb,greenplum-db/gpdb,adam8157/gpdb,xinzweb/gpdb,greenplum-db/gpdb,greenplum-db/gpdb,jmcatamney/gpdb,jmcatamney/gpdb,adam8157/gpdb,jmcatamney/gpdb,50wu/gpdb,adam8157/gpdb,50wu/gpdb,greenplum-db/gpdb,xinzweb/gpdb,ashwinstar/gpdb