[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. KUDU-2612: initial implementation of TxnManager This is a f

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 7: Code-Review+2 (6 comments) http://gerrit.cloudera.org:8080/

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 7: Verified+1 unrelated test failure in AdminCliTest.Tes

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Removed reviewer Kudu Jenkins with the following votes: * V

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#7). Change subject: KUDU-2612: initial implementation of TxnManager .

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#6). Change subject: KUDU-2612: initial implementation of TxnManager .

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 5: (9 comments) http://gerrit.cloudera.org:8080/#/c/1652

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/16527/

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/16527/5/sr

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 4: (11 comments) http://gerrit.cloudera.org:8080/#/c/165

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#5). Change subject: KUDU-2612: initial implementation of TxnManager .

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/16527/

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 4: Code-Review+1 (10 comments) Overall looks good, mostly

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#4). Change subject: KUDU-2612: initial implementation of TxnManager .

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16527 ) Change subject: KUDU-2612: initial implementation of TxnManager .. Patch Set 1: (16 comments) I split this patch in two, addressing t

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#3). Change subject: KUDU-2612: initial implementation of TxnManager .

[kudu-CR] KUDU-2612: initial implementation of TxnManager

2020-10-12 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16527 to look at the new patch set (#2). Change subject: KUDU-2612: initial implementation of TxnManager .