[ https://issues.apache.org/jira/browse/TRAFODION-3009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16441087#comment-16441087 ]
ASF GitHub Bot commented on TRAFODION-3009: ------------------------------------------- Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1522#discussion_r182136872 --- Diff: core/sql/executor/ExExeUtilCli.cpp --- @@ -1119,7 +1119,7 @@ Lng32 ExeCliInterface::executeImmediate(const char * stmtStr, { // Allocate the diagnostics area if needed // and populate the diagnostics conditions - if (*globalDiags == NULL && retcode != 0) { + if (*globalDiags == NULL && retcode != 0 && retcode != 100) { --- End diff -- I take it in the old code we would throw away the 100 warning somewhere else? > Streamline error handling in Executor utility commands > ------------------------------------------------------ > > Key: TRAFODION-3009 > URL: https://issues.apache.org/jira/browse/TRAFODION-3009 > Project: Apache Trafodion > Issue Type: Improvement > Components: -exe, sql-exe > Reporter: Selvaganesan Govindarajan > Assignee: Selvaganesan Govindarajan > Priority: Major > > Executor utility commands are the commands that issue sql statements during > its execution to complete its task. These operators are derived from > ExExeUtilTcb operator in Trafodion. These commands assume ComDiagsArea is > allocated in advance. This is not in accordance with the design as outlined in > [https://cwiki.apache.org/confluence/display/TRAFODION/SQL+Diagnostics+Architecture+and+Design] > -- This message was sent by Atlassian JIRA (v7.6.3#76005)