[ 
https://issues.apache.org/jira/browse/TRAFODION-2414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15798406#comment-15798406
 ] 

ASF GitHub Bot commented on TRAFODION-2414:
-------------------------------------------

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

    https://github.com/apache/incubator-trafodion/pull/885#discussion_r94589904
  
    --- Diff: core/sqf/inc/cextdecs/cextdecs.h ---
    @@ -289,6 +289,10 @@ _declspec(dllimport) short   DNUMOUT
     
     _declspec(dllimport) short   ENDTRANSACTION();
     
    +//errStr is allocated if errlen is not zero. Caller must deallocate errStr.
    +//Rest of functionality same as ENDTRANSACTION();
    +_declspec(dllimport) short   ENDTRANSACTION_ERR(char *&errStr, int 
&errlen);
    --- End diff --
    
    Thanks @prashanth-vasudev for introducing another api to deallocate the 
error message. It is not  clear why is it missing  in the second time 
declaration in this file. 


> 8616 error to display transaction conflict details.  
> -----------------------------------------------------
>
>                 Key: TRAFODION-2414
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2414
>             Project: Apache Trafodion
>          Issue Type: Improvement
>          Components: dtm, sql-exe
>    Affects Versions: any
>            Reporter: Prashanth Vasudev
>            Assignee: Prashanth Vasudev
>             Fix For: 2.1-incubating
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to