[
https://issues.apache.org/jira/browse/TRAFODION-2981?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16392312#comment-16392312
]
ASF GitHub Bot commented on TRAFODION-2981:
-------------------------------------------
Github user mashengchen commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1464#discussion_r173354026
--- Diff: core/sql/executor/ExExplain.cpp ---
@@ -1724,9 +1724,9 @@ short ExExplainTcb::getExplainFromRepos(char * qid,
Lng32 qidLen)
if (vi->get(0, ptr, len))
goto label_error2;
- explainFragLen_ = str_decoded_len(len); // remove trailing null
terminator
+ explainFragLen_ = str_decoded_len(len - 1); // remove trailing null
terminator
--- End diff --
for the '-1' , i think it should remove trailinng null terminator as the
comment.
but last time in anoop 's commit, the '-1' was removed, but the method
(getExplainFromRepos) is for EXPLAIN_QID, not for EXPLAIN_STMT (see
TRAFODION-1755), so i guess he changed the place there is no need to change.
> mxosrvr crashes during explain using EXPLAIN_QID
> ------------------------------------------------
>
> Key: TRAFODION-2981
> URL: https://issues.apache.org/jira/browse/TRAFODION-2981
> Project: Apache Trafodion
> Issue Type: Bug
> Reporter: mashengchen
> Assignee: mashengchen
> Priority: Major
>
> When the following sequence of commands are executed via trafci, the mxosrvr
> abends and we do not get the explain output.
> SELECT * FROM TABLE(explain(null,
> 'EXPLAIN_QID=MXID11000007019212384153894533416000000002206U3333308T150010100_923_SQL_CUR_2'));
>
> this issue is related to TRAFODION-1755
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
