[
https://issues.apache.org/jira/browse/TRAFODION-3166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16565932#comment-16565932
]
ASF GitHub Bot commented on TRAFODION-3166:
-------------------------------------------
Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1665#discussion_r206619328
--- Diff: core/sql/executor/ExSequence.cpp ---
@@ -992,23 +992,24 @@ short ExSequenceTcb::work()
break;
}
ex_assert(isUnboundedFollowing(),"");
-
- if ( ! cluster_->flush(&rc_) ) { // flush the buffers
+
+ ComDiagsArea *myDiags = NULL;
+ if (!cluster_->flush(myDiags, heap_)) { // flush the buffers
// if no errors this code path is not visited
- if ( rc_ )
+ if (myDiags)
--- End diff --
It is good to not to treat pointers as boolean variable
> 8427 error string missing when thrown by sequence operator
> ----------------------------------------------------------
>
> Key: TRAFODION-3166
> URL: https://issues.apache.org/jira/browse/TRAFODION-3166
> Project: Apache Trafodion
> Issue Type: Bug
> Components: sql-exe
> Reporter: Prashanth Vasudev
> Assignee: Prashanth Vasudev
> Priority: Major
> Fix For: 2.3
>
>
> *** ERROR[8427] [2018-07-23 15:52:16]
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
