[kudu-CR] KUDU-2039 Fix the table count in the /tables page of master webUI

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2039 Fix the table count in the /tables page of master webUI .. Patch Set 3: Code-Review+2 Thanks, looks good! -- To view, visit

[kudu-CR] KUDU-2039 Fix the table count in the /tables page of master webUI

2017-08-02 Thread Abhishek Talluri (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7568 to look at the new patch set (#3). Change subject: KUDU-2039 Fix the table count in the /tables page of master webUI

[kudu-CR] KUDU-2066. Add experimental Gradle build support

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 16: (14 comments) http://gerrit.cloudera.org:8080/#/c/7258/16/java/build.gradle File

[kudu-CR] KUDU-2039 Fix the table count in the /tables page of master webUI

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2039 Fix the table count in the /tables page of master webUI .. Patch Set 2: (1 comment) LINT build failed due to the lack of spaces between 'else' and '{'

[kudu-CR] Expose running maintenance op info

2017-08-02 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#5). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Add Maintenance Manager visualizer

2017-08-02 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7570 Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager visualizer Adds a timeline/swimlane chart to display

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#10). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 10: (3 comments) http://gerrit.cloudera.org:8080/#/c/7207/9/src/kudu/cfile/cfile-test.cc File

[kudu-CR] WIP KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#9). Change subject: WIP KUDU-1943: Add BlockTransaction to Block Manager .. WIP

[kudu-CR] KUDU-2066. Add experimental Gradle build support

2017-08-02 Thread Grant Henke (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7258 to look at the new patch set (#15). Change subject: KUDU-2066. Add experimental Gradle build support

[kudu-CR] Kudu-2060: Show primary keys in the master's table web UI page

2017-08-02 Thread Anonymous Coward (Code Review)
srisaikumarravip...@inspur.com has uploaded a new change for review. http://gerrit.cloudera.org:8080/7569 Change subject: Kudu-2060: Show primary keys in the master's table web UI page .. Kudu-2060: Show primary keys in the

[kudu-CR] KUDU-2039 Fix the table count in the /tables page of master webUI

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2039 Fix the table count in the /tables page of master webUI .. Patch Set 1: (3 comments)

[kudu-CR] consensus: always add backoff when snoozing failure detectors

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: consensus: always add backoff when snoozing failure detectors .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2039 Fix the table count in the /tables page of master webUI

2017-08-02 Thread Abhishek Talluri (Code Review)
Abhishek Talluri has uploaded a new change for review. http://gerrit.cloudera.org:8080/7568 Change subject: KUDU-2039 Fix the table count in the /tables page of master webUI .. KUDU-2039 Fix the table count in the /tables

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Move clock-related classes to src/kudu/clock .. Move clock-related classes to src/kudu/clock This creates a new module for clock-related classes

[kudu-CR] Kudu-2060: Show primary keys in the master's table web UI page

2017-08-02 Thread Anonymous Coward (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7566 to look at the new patch set (#2). Change subject: Kudu-2060: Show primary keys in the master's table web UI page

[kudu-CR] clock: refactor interaction with NTP into a new interface

2017-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: clock: refactor interaction with NTP into a new interface .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7520/5/src/kudu/clock/mock_ntp.cc File

[kudu-CR] Expose running maintenance op info

2017-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/7537/4/src/kudu/tserver/tserver-path-handlers.cc File

[kudu-CR] clock: refactor interaction with NTP into a new interface

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: clock: refactor interaction with NTP into a new interface .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7520/5/src/kudu/clock/mock_ntp.cc File

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Move clock-related classes to src/kudu/clock .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7519 To unsubscribe, visit

[kudu-CR] clock: refactor interaction with NTP into a new interface

2017-08-02 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7520 to look at the new patch set (#5). Change subject: clock: refactor interaction with NTP into a new interface ..

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7519 to look at the new patch set (#6). Change subject: Move clock-related classes to src/kudu/clock .. Move

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Move clock-related classes to src/kudu/clock .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/7519/4/src/kudu/clock/CMakeLists.txt File

[kudu-CR] Add metrics for tablet state and tablet copy

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add metrics for tablet state and tablet copy .. Patch Set 5: Code-Review+1 Plumbing looks good, but either Todd or Mike should also review this. -- To view,

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Move clock-related classes to src/kudu/clock .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7519/5/src/kudu/tablet/tablet_bootstrap-test.cc File

[kudu-CR] KUDU-2060: Show primary keys in the master's table web UI page

2017-08-02 Thread Anonymous Coward (Code Review)
srisaikumarravip...@inspur.com has abandoned this change. Change subject: KUDU-2060: Show primary keys in the master's table web UI page .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7555 To unsubscribe,

[kudu-CR] Kudu 2060: Show primary keys in the master's table web UI page

2017-08-02 Thread Anonymous Coward (Code Review)
srisaikumarravip...@inspur.com has uploaded a new change for review. http://gerrit.cloudera.org:8080/7566 Change subject: Kudu 2060: Show primary keys in the master's table web UI page .. Kudu 2060: Show primary keys in the

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-08-02 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7519 to look at the new patch set (#5). Change subject: Move clock-related classes to src/kudu/clock .. Move

[kudu-CR] kudu client tools for hadoop and spark import/export(csv,parquet,avro)

2017-08-02 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has posted comments on this change. Change subject: kudu client tools for hadoop and spark import/export(csv,parquet,avro) .. Patch Set 3: (2 comments)

[kudu-CR] clock: refactor interaction with NTP into a new interface

2017-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: clock: refactor interaction with NTP into a new interface .. Patch Set 4: (6 comments) I also realized that this patch will break builds on OSX so I'll do

[kudu-CR] kudu client tools for hadoop and spark import/export(csv,parquet,avro)

2017-08-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: kudu client tools for hadoop and spark import/export(csv,parquet,avro) .. Patch Set 3: (19 comments)

[kudu-CR] Add metrics for tablet state and tablet copy

2017-08-02 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7540 to look at the new patch set (#5). Change subject: Add metrics for tablet state and tablet copy .. Add metrics for