[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16470017#comment-16470017
 ] 

ASF GitHub Bot commented on IGNITE-7343:


Github user asfgit closed the pull request at:

https://github.com/apache/ignite/pull/3337


> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
>Priority: Major
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[Taras Ledkov (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454399#comment-16454399
 ] 

Taras Ledkov commented on IGNITE-7343:
--

[~vozerov], great comment. Thanks. Fixed at {{start}} and {{sendrRequest}} 
methods too.

> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
>Priority: Major
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[Taras Ledkov (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16409391#comment-16409391
 ] 

Taras Ledkov commented on IGNITE-7343:
--

[~vozerov], Thank a lot. Reconnect wasn't handled properly after merge with 
master.
Fixed. JDBC tests are OK.

> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
>Priority: Major
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[Taras Ledkov (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318503#comment-16318503
 ] 

Taras Ledkov commented on IGNITE-7343:
--

[~vozerov]
About *p.3* the {{ownThread}} is set on the enter to the method {{sendRequest}} 
and reset to *null* en exit from the method.
So, I guess connection pools will work properly.

> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



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

[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318373#comment-16318373
 ] 

ASF GitHub Bot commented on IGNITE-7343:


Github user tledkov-gridgain closed the pull request at:

https://github.com/apache/ignite/pull/3322


> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



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

[jira] [Commented] (IGNITE-7343) Add guard to prevent concurrent usage the JDBC thin connection

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/IGNITE-7343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318372#comment-16318372
 ] 

ASF GitHub Bot commented on IGNITE-7343:


GitHub user tledkov-gridgain opened a pull request:

https://github.com/apache/ignite/pull/3337

IGNITE-7343  Add guard to prevent concurrent usage the JDBC thin connection



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gridgain/apache-ignite ignite-7343

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/ignite/pull/3337.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #3337


commit 2fe1ca63a08ef3c12f239abfb5549b5a57096447
Author: tledkov-gridgain 
Date:   2017-12-29T12:43:26Z

IGNITE-7335: add multi-tread guard

commit 227841f0f7bb5169d5e995705ac4dcf98e9dde86
Author: tledkov-gridgain 
Date:   2018-01-09T12:44:43Z

Merge branch '_master' into ignite-7335




> Add guard to prevent concurrent usage the JDBC thin connection
> --
>
> Key: IGNITE-7343
> URL: https://issues.apache.org/jira/browse/IGNITE-7343
> Project: Ignite
>  Issue Type: Task
>  Components: jdbc
>Affects Versions: 2.3
>Reporter: Taras Ledkov
>Assignee: Taras Ledkov
> Fix For: 2.5
>
>
> I propose to add multi-thread guard into {{JdbcThinTcpIo#sendRequest}} method 
> to diagnostic concurrency usage of the Ignite JDBC connection. 
> The related ticket: IGNITE-7335



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