Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1665#discussion_r205606604
--- Diff: core/sql/executor/ExSequence.cpp ---
@@ -1487,7 +1487,35 @@ void ExSequenceTcb::updateDiagsArea( ExeErrorCode
rc_)
}
if (!da->contains((Lng32) -rc_))
{
- *da << DgSqlCode(-rc_);
+ char msg[512];
--- End diff --
It would be nice if you can encapsulate these changes in cluster_->flush().
The caller doesn't need to know about the internals of cluster_---
