[kudu-CR] Clean up spurious declarations in messenger.cc

2017-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Clean up spurious declarations in messenger.cc .. Clean up spurious declarations in messenger.cc Change-Id: I105524bab85bef9c08205e3d1f262fa5234745d9

[kudu-CR] Clean up spurious declarations in messenger.cc

2017-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Clean up spurious declarations in messenger.cc .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7583 To unsubscribe, visit

[kudu-CR] Test Prepare() failed codepath

2017-08-04 Thread Sam Okrent (Code Review)
Sam Okrent has abandoned this change. Change subject: Test Prepare() failed codepath .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7522 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7570 to look at the new patch set (#6). Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager

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

2017-08-04 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 (#13). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] log block manager: Reorder class declaration of LogWritableBlock

2017-08-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: log block manager: Reorder class declaration of LogWritableBlock .. log block manager: Reorder class declaration of LogWritableBlock Reordered class declaration of

[kudu-CR] log block manager: Reorder class declaration of LogWritableBlock

2017-08-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7595 Change subject: log block manager: Reorder class declaration of LogWritableBlock .. log block manager: Reorder class declaration of

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add Maintenance Manager visualizer .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7570/5/www/maintenance-manager.mustache File

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7570 to look at the new patch set (#5). Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager

[kudu-CR] [tests] fix compilation warning on OS X

2017-08-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [tests] fix compilation warning on OS X .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7594 To unsubscribe, visit

[kudu-CR] [tests] fix compilation warning on OS X

2017-08-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7594 Change subject: [tests] fix compilation warning on OS X .. [tests] fix compilation warning on OS X Fixed compilation warning on OS X

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 8: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7537 To unsubscribe, visit

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

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

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

2017-08-04 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 (#12). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] WIP [thirdparty]: added include-what-you-use

2017-08-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7593 Change subject: WIP [thirdparty]: added include-what-you-use .. WIP [thirdparty]: added include-what-you-use Build the

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Add Maintenance Manager visualizer .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7570/3//COMMIT_MSG Commit Message: PS3, Line 7: Add

[kudu-CR] Expose running maintenance op info

2017-08-04 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 (#8). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

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

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 7: (2 comments) Couple more nits.

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

2017-08-04 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 5: (1 comment)

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

2017-08-04 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: (3 comments)

[kudu-CR] Expose running maintenance op info

2017-08-04 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 (#7). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] [build] Fix Python build failures on RHEL 6

2017-08-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [build] Fix Python build failures on RHEL 6 .. [build] Fix Python build failures on RHEL 6 Internal build machines running RHEL 6 started failing

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/7537/5/src/kudu/util/maintenance_manager.cc File

[kudu-CR] [build] Fix Python build failures on RHEL 6

2017-08-04 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [build] Fix Python build failures on RHEL 6 .. Patch Set 1: I'll keep looking for a "non-workaround" version. -- To view, visit

[kudu-CR] Expose running maintenance op info

2017-08-04 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 (#6). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] [build] Fix Python build failures on RHEL 6

2017-08-04 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [build] Fix Python build failures on RHEL 6 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7589 To unsubscribe, visit

[kudu-CR] Revert "Kudu-client ImportCsv small fix's"

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new patch set (#2). Change subject: Revert "Kudu-client ImportCsv small fix's" .. Revert "Kudu-client ImportCsv small fix's" This reverts commit 6163c0e1442a92f9029de26d1298a4d2cd50ec69.

[kudu-CR] Revert "Kudu-client ImportCsv small fix's"

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has abandoned this change. Change subject: Revert "Kudu-client ImportCsv small fix's" .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7586 To unsubscribe, visit

[kudu-CR] Revert "Kudu-client ImportCsv small fix's"

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new change for review. http://gerrit.cloudera.org:8080/7586 Change subject: Revert "Kudu-client ImportCsv small fix's" .. Revert "Kudu-client ImportCsv small fix's" This reverts commit

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has abandoned this change. Change subject: Kudu-client ImportCsv small fix's .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7585 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7585 to look at the new patch set (#3). Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has restored this change. Change subject: Kudu-client ImportCsv small fix's .. Restored -- To view, visit http://gerrit.cloudera.org:8080/7585 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7584 to look at the new patch set (#4). Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7584 to look at the new patch set (#3). Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new patch set (#2). Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv small fix's Change-Id: I422e83d99db38be7a011ddc00b3412d542e2c181 --- M

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has abandoned this change. Change subject: Kudu-client ImportCsv small fix's .. Abandoned NA -- To view, visit http://gerrit.cloudera.org:8080/7585 To unsubscribe, visit

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new patch set (#2). Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv small fix's Change-Id: I54ab38cd00d4aba4d899584e446e9c6e11217550 --- D

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new change for review. http://gerrit.cloudera.org:8080/7585 Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv small fix's Change-Id:

[kudu-CR] Kudu-client ImportCsv small fix's

2017-08-04 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded a new change for review. http://gerrit.cloudera.org:8080/7584 Change subject: Kudu-client ImportCsv small fix's .. Kudu-client ImportCsv small fix's Change-Id:

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

2017-08-04 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 (#11). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943: