Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1464#discussion_r174177517
  
    --- 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 --
    
    @sandhyasun, are you content with this change now? Can I merge it? 


---

Reply via email to