Fix low probability hanging in diff-tree execution

grundprinzip / gerrit-check

Line annotations for syntax checkers for Gerrit

Apache License 2.0

15 stars 9 forks source link

Fix low probability hanging in diff-tree execution #8

Closed Jimilian closed 7 years ago

Jimilian commented 7 years ago

Chance to see this issue in real life is quite small. Only one developer in our company was lucky enough to make change set that is able to repeat deadlock in subprocess.wait module in 100% cases.

More details in: https://github.com/tomerfiliba/plumbum/issues/295

Jimilian commented 7 years ago

@grundprinzip, can you take a look, please? :)

grundprinzip commented 7 years ago

Sorry for the delay!

On Tue, Dec 27, 2016 at 10:28 AM Akbashev Alexander < notifications@github.com> wrote:

@grundprinzip https://github.com/grundprinzip, can you take a look, please? :)

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/grundprinzip/gerrit-check/pull/8#issuecomment-269300308, or mute the thread https://github.com/notifications/unsubscribe-auth/AAANXYDjgWWCgpFARLEVGfSRjSBbQRi-ks5rMNo-gaJpZM4KVsqV .