Remove now-unnecessary code from gp_read_error_log to dispatch the call.
Remove now-unnecessary code from gp_read_error_log to dispatch the call.

There was code in gp_read_error_log(), to "manually" dispatch the call to
all the segments, if it was executed in the dispatcher. This was
previously necessary, because even though the function was marked with
prodataaccess='s', the planner did not guarantee that it's executed in the
segments, when called in the targetlist like "SELECT
gp_read_error_log('tab')". Now that we have the EXECUTE ON ALL SEGMENTS
syntax, and are more rigorous about enforcing that in the planner, this
hack is no longer required.

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