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

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

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

    
https://github.com/apache/incubator-trafodion/pull/1314#discussion_r153566118
  
    --- Diff: core/sql/executor/SequenceFileReader.cpp ---
    @@ -189,9 +192,10 @@ SFR_RetCode SequenceFileReader::open(const char* path)
       if (jresult != NULL)
       {
         logError(CAT_SQL_HDFS_SEQ_FILE_READER, "SequenceFileReader::open()", 
jresult);
    +    jenv_->PopLocalFrame(NULL);
    --- End diff --
    
    The DeleteLocalRef at line 190 is no longer needed, but harmless, right?


> Trafodion core code base needs to be thread safe 
> -------------------------------------------------
>
>                 Key: TRAFODION-2821
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2821
>             Project: Apache Trafodion
>          Issue Type: Improvement
>          Components: sql-general
>    Affects Versions: any
>            Reporter: Selvaganesan Govindarajan
>            Assignee: Selvaganesan Govindarajan
>             Fix For: 2.3-incubating
>
>
> This is the covering trafodion jira to make trafodion core code base to be 
> thread safe. It is needed to ensure that type T2 JDBC driver hosted on 
> platform can be enabled to support multi-threaded JDBC applications.
> [TRAFODION-2783] is one such case.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to