[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16952 ) Change subject: KUDU-2612: background task to commit transaction .. Patch Set 16: (1 comment) Ah oops, clicked submit without another r

[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16952 ) Change subject: KUDU-2612: background task to commit transaction .. KUDU-2612: background task to commit transaction This patch

[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16952 ) Change subject: KUDU-2612: background task to commit transaction .. Patch Set 15: Code-Review+2 (5 comments) Looks good to me! Maybe

[kudu-CR] [master] turn off client location assignment by default

2021-02-04 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17024 to look at the new patch set (#2). Change subject: [master] turn off client location assignment by default ...

[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16952 ) Change subject: KUDU-2612: background task to commit transaction .. Patch Set 15: (8 comments) http://gerrit.cloudera.org:8080/#/c/1695

[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16952 to look at the new patch set (#15). Change subject: KUDU-2612: background task to commit transaction ...

[kudu-CR] [master] turn off client location assignment by default

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17024 ) Change subject: [master] turn off client location assignment by default .. Patch Set 1: > Build Failed > > http://jenkins.kudu.apach

[kudu-CR] [master] turn off client location assignment by default

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17024 Change subject: [master] turn off client location assignment by default .. [master] turn off client location assignment by default This

[kudu-CR] KUDU-2612: background task to commit transaction

2021-02-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16952 ) Change subject: KUDU-2612: background task to commit transaction .. Patch Set 14: (9 comments) http://gerrit.cloudera.org:8080/#/c/16952/14

[kudu-CR] [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances

2021-02-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/17021 ) Change subject: [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances .. Patch Set 4: > Patch Set 3: Code-Review+2

[kudu-CR] [tool] KUDU-2181 CLI to add master

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16530 ) Change subject: [tool] KUDU-2181 CLI to add master .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/16530/2/src/kudu/

[kudu-CR] [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17021 ) Change subject: [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances .. [test] enable TxnStatusManag

[kudu-CR] [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances

2021-02-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17021 ) Change subject: [test] enable TxnStatusManagerITest.TxnKeepAliveMultiTxnStatusManagerInstances .. Patch Set 3: Code-Review+2 Thank yo