[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-19 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on IGNITE-10285:
-

Github user asfgit closed the pull request at:

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


> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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


[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-19 Thread Dmitriy Govorukhin (JIRA)


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

Dmitriy Govorukhin commented on IGNITE-10285:
-

[~voropava], [~Jokser] Thanks for the review, merged to master.

> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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


[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-19 Thread Pavel Kovalenko (JIRA)


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

Pavel Kovalenko commented on IGNITE-10285:
--

[~DmitriyGovorukhin] Thank you for contribution. Changes look good to me.

> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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


[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-19 Thread Ignite TC Bot (JIRA)


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

Ignite TC Bot commented on IGNITE-10285:


{panel:title=No blockers 
found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}{panel}
[TeamCity Run All 
Results|https://ci.ignite.apache.org/viewLog.html?buildId=2346600&buildTypeId=IgniteTests24Java8_RunAll]

> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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


[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-16 Thread Pavel Voronkin (JIRA)


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

Pavel Voronkin commented on IGNITE-10285:
-

Looks good for me.

> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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


[jira] [Commented] (IGNITE-10285) U.doInParallel may lead to deadlock

2018-11-15 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on IGNITE-10285:
-

GitHub user dgovorukhin opened a pull request:

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

IGNITE-10285 Implement job stealing for U.doInParallel 



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

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

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

https://github.com/apache/ignite/pull/5404.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 #5404


commit 37d1310d24996966c60d673754033051f02abccb
Author: Dmitriy Govorukhin 
Date:   2018-11-15T22:37:40Z

IGNITE-10285 Implement job stealing for U.doInParallel in case if executor 
busy and cannot perform task immediately

Signed-off-by: Dmitriy Govorukhin 




> U.doInParallel may lead to deadlock
> ---
>
> Key: IGNITE-10285
> URL: https://issues.apache.org/jira/browse/IGNITE-10285
> Project: Ignite
>  Issue Type: Bug
>Reporter: Dmitriy Govorukhin
>Assignee: Dmitriy Govorukhin
>Priority: Critical
> Fix For: 2.8
>
> Attachments: dump.rtf
>
>
> There are exist case when we can get a deadlock on the thread pool.
> If we try doInParallel in the thread of sys-pool in the number of 
> hreads==sys-pool.size we lead to deadlock because threads in sys-pool will 
> try doInParallel through the same sys-pool, and they will wait on future 
> infinitely because no one thread cannot complete operation doInParallel which 
> require threads from sys-pool.



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