[kudu-CR] rpc: hook up a callback for libev fatal errors

2017-08-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: rpc: hook up a callback for libev fatal errors .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7633 To unsubscribe, visit

[kudu-CR] [java] Remove double brackets from wrapper script

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java] Remove double brackets from wrapper script .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7655 To unsubscribe, visit

[kudu-CR] [java] Remove double brackets from wrapper script

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [java] Remove double brackets from wrapper script .. [java] Remove double brackets from wrapper script When running on Jenkins the gradle wrapper script

[kudu-CR] [WIP] Add BlockDeletionTransaction to Block Manager

2017-08-10 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7656 Change subject: [WIP] Add BlockDeletionTransaction to Block Manager .. [WIP] Add BlockDeletionTransaction to Block Manager Similar to

[kudu-CR] [maintenance manager] fix op scheduling lock contention, thread ID tweak

2017-08-10 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [maintenance manager] fix op scheduling lock contention, thread ID tweak .. [maintenance manager] fix op scheduling lock contention, thread ID tweak

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-08-10 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6968 to look at the new patch set (#9). Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric ..

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-08-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric .. Patch Set 6: (1 comment) Minor rebasing.

[kudu-CR] KUDU-2078: Sink failure if batch size > session's flush buffer size

2017-08-10 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/7641 Change subject: KUDU-2078: Sink failure if batch size > session's flush buffer size .. KUDU-2078: Sink failure if batch size >

[kudu-CR] [java] Fix spark-avro scala binary version

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7644 Change subject: [java] Fix spark-avro scala binary version .. [java] Fix spark-avro scala binary version Fixes the spark-avro scala

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: Code-Review+2 > I ran 'gradle clean assemble` to make sure this fixed compile > errors.

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java] Sync gradle build with dependecy changes .. [java] Sync gradle build with dependecy changes Adds the newly added parquet and spark-avro

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit

[kudu-CR] Allow tablet shutdown without completing txs

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7439 to look at the new patch set (#9). Change subject: Allow tablet shutdown without completing txs .. Allow tablet

[kudu-CR] Allow tablet shutdown without completing txs

2017-08-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Allow tablet shutdown without completing txs .. Patch Set 8: (7 comments) Also as per our slack discussion, rather than making use of tablet::FAILED, which in

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7649 Change subject: [gutil] remove the rest of 'using ...' from headers .. [gutil] remove the rest of 'using ...' from headers Removed the

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: [gutil] remove the rest of 'using ...' from headers .. [gutil] remove the rest of 'using ...' from headers Removed the rest of 'using ...' directives from gutil

[kudu-CR] [cfile-test-base.h] remove cyclic dependency

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7650 Change subject: [cfile-test-base.h] remove cyclic dependency .. [cfile-test-base.h] remove cyclic dependency Removed '#include

[kudu-CR] [cfile-test-base.h] remove cyclic dependency

2017-08-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [cfile-test-base.h] remove cyclic dependency .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7650 To unsubscribe, visit

[kudu-CR] Run the gradle build as a part of the gerrit tests

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7651 Change subject: Run the gradle build as a part of the gerrit tests .. Run the gradle build as a part of the gerrit tests Change-Id:

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [gutil] remove the rest of 'using ...' from headers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7649 To unsubscribe, visit

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#16). Change subject: Hive MetaStore client, HMS Plugin .. Hive MetaStore client,

[kudu-CR] Run the gradle build as a part of the gerrit tests

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: Run the gradle build as a part of the gerrit tests .. Run the gradle build as a part of the gerrit tests Change-Id: I1430cbd05ff78a69f2439e3a8f90e1ddde83a8d7 --- M

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#17). Change subject: Hive MetaStore client, HMS Plugin .. Hive MetaStore client,

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Hive MetaStore client, HMS Plugin .. Patch Set 16: (6 comments) http://gerrit.cloudera.org:8080/#/c/7053/12/src/kudu/hms/hms_client-test.cc File

[kudu-CR] [cfile-test-base.h] remove cyclic dependency

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [cfile-test-base.h] remove cyclic dependency .. [cfile-test-base.h] remove cyclic dependency Removed '#include "kudu/cfile/cfile-test-base.h"' from the

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7649 to look at the new patch set (#3). Change subject: [gutil] remove the rest of 'using ...' from headers

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

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

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

2017-08-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 14: (17 comments) http://gerrit.cloudera.org:8080/#/c/7207/13//COMMIT_MSG Commit Message: PS13, Line

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7649 to look at the new patch set (#4). Change subject: [gutil] remove the rest of 'using ...' from headers

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [gutil] remove the rest of 'using ...' from headers .. Patch Set 3: (8 comments) http://gerrit.cloudera.org:8080/#/c/7649/3/src/kudu/gutil/hash/hash.h File

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [gutil] remove the rest of 'using ...' from headers .. Patch Set 4: Carrying over Dan's +2 from PS2. -- To view, visit http://gerrit.cloudera.org:8080/7649 To

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [gutil] remove the rest of 'using ...' from headers .. Patch Set 4: Code-Review+2 Carrying over Dan's +2 from PS2. -- To view, visit

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#18). Change subject: Hive MetaStore client, HMS Plugin .. Hive MetaStore client,

[kudu-CR] [gutil] remove the rest of 'using ...' from headers

2017-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [gutil] remove the rest of 'using ...' from headers .. [gutil] remove the rest of 'using ...' from headers Removed the rest of 'using ...' directives

[kudu-CR] Web UI: switch /maintenance-manager endpoint to mustache

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Web UI: switch /maintenance-manager endpoint to mustache .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7607 To

[kudu-CR] Web UI: switch /maintenance-manager endpoint to mustache

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Web UI: switch /maintenance-manager endpoint to mustache .. Web UI: switch /maintenance-manager endpoint to mustache This cherry-picks just the

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7645 Change subject: [java] Sync gradle build with dependecy changes .. [java] Sync gradle build with dependecy changes Adds the newly added

[kudu-CR] [java] Update outdated dependencies

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7647 Change subject: [java] Update outdated dependencies .. [java] Update outdated dependencies Includes the following version updates: - Avro

[kudu-CR] rpc: hook up a callback for libev fatal errors

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: rpc: hook up a callback for libev fatal errors .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7633 To unsubscribe, visit

[kudu-CR] transaction: remove unused StatusTransactionCompletionCallback

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: transaction: remove unused StatusTransactionCompletionCallback .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7632 To

[kudu-CR] WIP: clock: add a built-in NTP client implementation

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: clock: add a built-in NTP client implementation .. Patch Set 5: (1 comment) only browsed this briefly. The direction looks good, but would like to

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc File

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#15). Change subject: Hive MetaStore client, HMS Plugin .. Hive MetaStore client,

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc File

[kudu-CR] KUDU-871. Support tombstoned voting

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-871. Support tombstoned voting .. Patch Set 6: (7 comments) leaving for luch, more comments forthcoming

[kudu-CR] transaction: remove unused StatusTransactionCompletionCallback

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: transaction: remove unused StatusTransactionCompletionCallback .. transaction: remove unused StatusTransactionCompletionCallback This has been unused

[kudu-CR] [java] Update outdated dependencies

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: [java] Update outdated dependencies .. [java] Update outdated dependencies Includes the following version updates: - Avro 1.8.1 -> 1.8.2 - Flume 1.6.0 -> 1.7.0 -

[kudu-CR] [java] Fix spark-avro scala binary version

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Fix spark-avro scala binary version .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7644 To unsubscribe, visit

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#9). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign

[kudu-CR] rpc: hook up a callback for libev fatal errors

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: rpc: hook up a callback for libev fatal errors .. rpc: hook up a callback for libev fatal errors In troubleshooting a recent cluster issue, I found that

[kudu-CR] [java] Remove double brackets from wrapper script

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7655 Change subject: [java] Remove double brackets from wrapper script .. [java] Remove double brackets from wrapper script When running on

[kudu-CR] WIP: support for running Java tests in dist-test

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: support for running Java tests in dist-test .. Patch Set 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/7579/1//COMMIT_MSG Commit Message: Line 10:

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7620 To unsubscribe, visit

[kudu-CR] [java] Update outdated dependencies

2017-08-10 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7647 to look at the new patch set (#3). Change subject: [java] Update outdated dependencies .. [java] Update outdated

[kudu-CR] [java] Update outdated dependencies

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Update outdated dependencies .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7647/2/java/pom.xml File java/pom.xml: Line 32:

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7645/1//COMMIT_MSG Commit Message: Line

[kudu-CR] Allow tablet shutdown without completing txs

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7439 to look at the new patch set (#11). Change subject: Allow tablet shutdown without completing txs .. Allow tablet

[kudu-CR] [java] Sync gradle build with dependecy changes

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: I ran 'gradle clean assemble` to make sure this fixed compile errors. The Jenkins failures appear

[kudu-CR] KUDU-871. Support tombstoned voting

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-871. Support tombstoned voting .. Patch Set 6: (25 comments) http://gerrit.cloudera.org:8080/#/c/6960/6/src/kudu/consensus/raft_consensus.cc File

[kudu-CR] Hive MetaStore client, HMS Plugin

2017-08-10 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#19). Change subject: Hive MetaStore client, HMS Plugin .. Hive MetaStore client,

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

2017-08-10 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout .. Patch Set 2: (33 comments) http://gerrit.cloudera.org:8080/#/c/7599/1//COMMIT_MSG Commit

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

2017-08-10 Thread Michael Ho (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7599 to look at the new patch set (#2). Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

[kudu-CR] [java] Fix spark-avro scala binary version

2017-08-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java] Fix spark-avro scala binary version .. [java] Fix spark-avro scala binary version Fixes the spark-avro scala binary version to be 2.11 and

[kudu-CR] [java] Update outdated dependencies

2017-08-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Update outdated dependencies .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7647/2/java/pom.xml File java/pom.xml: Line 32:

[kudu-CR] KUDU-871. Support tombstoned voting

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-871. Support tombstoned voting .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/6960/6/src/kudu/tablet/tablet_replica.h File

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: Code-Review+2 sounds good -- To view, visit http://gerrit.cloudera.org:8080/7620 To

[kudu-CR] Allow tablet shutdown without completing txs

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7439 to look at the new patch set (#10). Change subject: Allow tablet shutdown without completing txs .. Allow tablet

[kudu-CR] disk failure: tests for disk failure recovery

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7031 to look at the new patch set (#11). Change subject: disk failure: tests for disk failure recovery .. disk failure:

[kudu-CR] disk failure: local testing for disk failure

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7441 to look at the new patch set (#4). Change subject: disk failure: local testing for disk failure .. disk failure:

[kudu-CR] disk failure: shutdown tablets on disk failure

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7442 to look at the new patch set (#4). Change subject: disk failure: shutdown tablets on disk failure .. disk failure:

[kudu-CR] handle disk failures during tablet copies

2017-08-10 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7654 Change subject: handle disk failures during tablet copies .. handle disk failures during tablet copies There are two components to tablet

[kudu-CR] disk failure: tests for disk failure recovery

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7031 to look at the new patch set (#12). Change subject: disk failure: tests for disk failure recovery .. disk failure:

[kudu-CR] disk failure: add persistent disk states

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#8). Change subject: disk failure: add persistent disk states .. disk failure: add

[kudu-CR] disk failure: add persistent disk states

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#9). Change subject: disk failure: add persistent disk states .. disk failure: add

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc

[kudu-CR] Run the gradle build as a part of the gerrit tests

2017-08-10 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7651 to look at the new patch set (#4). Change subject: Run the gradle build as a part of the gerrit tests .. Run the

[kudu-CR] Run the gradle build as a part of the gerrit tests

2017-08-10 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7651 to look at the new patch set (#5). Change subject: Run the gradle build as a part of the gerrit tests .. Run the