[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894 fixed deadlock in client.Connection Due to the reverse order of acquiring of the

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 5: > This looks good to me, but obviously this code is extremely > fragile. We should figure out a

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: This looks good to me, but obviously this code is extremely fragile. We should figure out a way to

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (5 comments)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#5). Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894 fixed deadlock in client.Connection Due to the reverse order of acquiring of the Connection.lock and

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: (5 comments)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 4: Verified+1 unrelated flakes in MiniTabletServerTest.TestMultiDirServer

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7765 to look at the new patch set (#4). Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-23 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7765 to look at the new patch set (#3). Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: Code-Review-1 (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894 fixed deadlock in client.Connection Due to the reverse order of acquiring the Connection.lock and

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/7765/1//COMMIT_MSG Commit Message: Line 13: