[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-27 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. KUDU-2612 Java client transaction implementation This

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 9: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: KUDU-2612 Java client transaction implementation .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16929 To

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 9: Verified+1 Unrelated test failure in TSAN build:

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#9). Change subject: KUDU-2612 Java client transaction implementation

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 8: Code-Review+1 (2 comments) Overall looks good to me,

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 8: Verified+1 Unrelated build issue in TSAN build:

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: KUDU-2612 Java client transaction implementation .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16929 To

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-26 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#8). Change subject: KUDU-2612 Java client transaction implementation

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#7). Change subject: KUDU-2612 Java client transaction implementation

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#6). Change subject: KUDU-2612 Java client transaction implementation

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 5: (2 comments) Overall looks fine -- just one more

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: (7 comments)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#5). Change subject: KUDU-2612 Java client transaction implementation

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: (5 comments)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: (5 comments)

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: > I will look into this gradle

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: I will look into this gradle

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: That was even build failure for

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Patch Set 4: Verified+1 unrelated test failure in ASAN build --

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/16929 ) Change subject: KUDU-2612 Java client transaction implementation .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-18 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16929 to look at the new patch set (#4). Change subject: KUDU-2612 Java client transaction implementation