[
https://issues.apache.org/jira/browse/TRAFODION-2821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16473784#comment-16473784
]
ASF GitHub Bot commented on TRAFODION-2821:
-------------------------------------------
Github user xiaozhongwang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1562#discussion_r187841031
--- Diff: core/sqf/src/tm/tmtransaction.cpp ---
@@ -797,7 +797,7 @@ short TM_Transaction::status(short *pp_status)
return iv_last_error;
}
- if (!gv_tmlib.is_initialized())
+ if (!gv_tmlib_initialized)
--- End diff --
There are a lot of same code place as this:
if (!gv_tmlib_initialized)
gv_tmlib.initialize();
I think it's better to add a method to do the same thing. but some of the
places are in order to performance. just a suggestion.
> 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
> Priority: Major
> Fix For: 2.3
>
>
> 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
(v7.6.3#76005)
