Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1380#discussion_r161272011
--- Diff: core/sql/executor/cluster.cpp ---
@@ -2395,7 +2395,7 @@ NABoolean Cluster::checkAndSplit(ExeErrorCode * rc)
rc);
if ( !next_ || *rc ) {
--- End diff --
You are correct, In that case you can change it if *(!next) || (*rc !=
EXE_OK)). I wish we got this right this time.---
