[kudu-CR] KUDU-1883. Support Ruby 3.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1883. Support Ruby 3.4 in make docs task .. Patch Set 2: I manually verified the build still works on rhel6.7 and ubuntu14.04. -- To view, visit

[kudu-CR] KUDU-1883. Support Ruby 3.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6022 to look at the new patch set (#3). Change subject: KUDU-1883. Support Ruby 3.4 in make docs task .. KUDU-1883.

[kudu-CR] KUDU-1883. Support Ruby 2.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6022 to look at the new patch set (#4). Change subject: KUDU-1883. Support Ruby 2.4 in make docs task .. KUDU-1883.

[kudu-CR] KUDU-1883. Support Ruby 3.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1883. Support Ruby 3.4 in make docs task .. Patch Set 2: I verified that the changes still work with older versions of Ruby but did not validate each

[kudu-CR] Fix checkstyle suppressions in submodule builds

2017-02-20 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Fix checkstyle suppressions in submodule builds .. Patch Set 4: (1 comment) I did a sanity check by running a top level build and a few submodule builds and

[kudu-CR] Fix checkstyle suppressions in submodule builds

2017-02-20 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Fix checkstyle suppressions in submodule builds .. Patch Set 4: I did not need to pass -DbinDir to run tests from the submodules. Since I am still in the Kudu

[kudu-CR] Fix checkstyle suppressions in submodule builds

2017-02-18 Thread Grant Henke (Code Review)
Hello Jean-Daniel Cryans, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6060 to look at the new patch set (#4). Change subject: Fix checkstyle suppressions in submodule builds .. Fix

[kudu-CR] Fix checkstyle suppressions in submodule builds

2017-02-17 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6060 to look at the new patch set (#3). Change subject: Fix checkstyle suppressions in submodule builds .. Fix

[kudu-CR] Fix checkstlye suppressions in submodule builds

2017-02-17 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6060 Change subject: Fix checkstlye suppressions in submodule builds .. Fix checkstlye suppressions in submodule builds Removes the use of

[kudu-CR] KUDU-1402. Ensure session.flush() returns an empty list

2017-02-17 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6061 Change subject: KUDU-1402. Ensure session.flush() returns an empty list .. KUDU-1402. Ensure session.flush() returns an empty list Adds a

[kudu-CR] Fix checkstlye suppressions in submodule builds

2017-02-17 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: Fix checkstlye suppressions in submodule builds .. Fix checkstlye suppressions in submodule builds Removes the use of relative paths via “samedir=.” in the

[kudu-CR] Minor updates to the installation from source docs

2017-02-16 Thread Grant Henke (Code Review)
Hello Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6041 to look at the new patch set (#2). Change subject: Minor updates to the installation from source docs .. Minor

[kudu-CR] Fix checkstyle suppressions in submodule builds

2017-02-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Fix checkstyle suppressions in submodule builds .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6060/1//COMMIT_MSG Commit Message: PS1, Line 7:

[kudu-CR] Minor updates to the installation from source docs

2017-02-16 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6041 Change subject: Minor updates to the installation from source docs .. Minor updates to the installation from source docs A few small

[kudu-CR] KUDU-1883. Support Ruby 3.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6022 Change subject: KUDU-1883. Support Ruby 3.4 in make docs task .. KUDU-1883. Support Ruby 3.4 in make docs task Bumps the json gem to

[kudu-CR] KUDU-1883. Support Ruby 3.4 in make docs task

2017-02-15 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: KUDU-1883. Support Ruby 3.4 in make docs task .. KUDU-1883. Support Ruby 3.4 in make docs task Bumps the json gem to 1.8.5 in order to support ruby 3.4.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#7). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 7: I made a few layout changes and published the generated docs here:

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 1: This is just a quick push to sanity check my approach. -- To view, visit

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6630 Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add checksumming to cfile Adds optional checksumming and

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#2). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-20 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#6). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-19 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#5). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-20 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/6630/6/src/kudu/cfile/cfile_reader.cc File

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#9). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 8: > Any way to reduce the amount of whitespace between commands in the > "command hierarchy" section of

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/6525/8/src/kudu/tools/CMakeLists.txt File

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 8: > Any way to reduce the amount of whitespace between commands in the > "command hierarchy" section of

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#8). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-02 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 3: (9 comments) I will publish the generated docs in a GH branch tomorrow. (For some reason the

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-02 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#5). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-02 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#4). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-04-02 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#6). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-03-31 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#2). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-03-31 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6525 to look at the new patch set (#3). Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708.

[kudu-CR] KUDU-1708. Document Kudu command-line tools

2017-03-31 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6525 Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708. Document Kudu command-line tools - Adds helpxml flag support to

[kudu-CR] KUDU-1424. Add getters to PartialRow

2017-04-07 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1424. Add getters to PartialRow .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1424. Add getters to PartialRow

2017-04-07 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6554 to look at the new patch set (#2). Change subject: KUDU-1424. Add getters to PartialRow .. KUDU-1424. Add getters

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 3: I need to handle the header and footer yet, which may require a change in how to handle backwards

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-14 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#3). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-18 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#4). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR](branch-1.3.x) KUDU-1708. Document Kudu command-line tools

2017-04-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6645/2/src/kudu/tools/tool_action.cc File

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 3: (9 comments) http://gerrit.cloudera.org:8080/#/c/6630/3//COMMIT_MSG Commit Message: Line 19: immeditaly

[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 (#16). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-08-02 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 14: (46 comments)

[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-2066. Add experimental Gradle build support

2017-08-03 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 (#17). Change subject: KUDU-2066. Add experimental Gradle build support

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

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

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

2017-08-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/7258/17/java/build.gradle File

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

2017-08-03 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 (#18). Change subject: KUDU-2066. Add experimental Gradle build support

[kudu-CR] [java] Update outdated dependencies

2017-08-15 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 (#6). Change subject: [java] Update outdated dependencies .. [java] Update outdated

[kudu-CR] [java] Update outdated dependencies

2017-08-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Update outdated dependencies .. Patch Set 6: Turns out Spark 2.2 dropped java 7 support. I will open a discussion to see how we want to handle that on the

[kudu-CR] [java] Update outdated dependencies

2017-08-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Update outdated dependencies .. Patch Set 7: I removed the Spark 2.2 upgrade since I found it only supports Java 8. I will handle that separately since it

[kudu-CR] [java] Update outdated dependencies

2017-08-15 Thread Grant Henke (Code Review)
Hello Adar Dembo, 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 (#7). Change subject: [java] Update outdated dependencies .. [java]

[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] 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] 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] [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] [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] Remove double brackets from wrapper script

2017-08-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Remove double brackets from wrapper script .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7655/2//COMMIT_MSG Commit Message: Line 13:

[kudu-CR] [java] Update outdated dependencies

2017-08-11 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 (#5). Change subject: [java] Update outdated dependencies .. [java] Update outdated

[kudu-CR] [java] Update outdated dependencies

2017-08-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] Update outdated dependencies .. Patch Set 5: Yeah, they are absolutely legitimate test errors. It looks like another issue with shading order since it

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

2017-08-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Run the gradle build as a part of the gerrit tests .. Patch Set 5: (3 comments)

[kudu-CR] [docs] Deprecate Java 7 and Spark 1

2017-08-16 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7699 Change subject: [docs] Deprecate Java 7 and Spark 1 .. [docs] Deprecate Java 7 and Spark 1 Starts the release notes for Kudu 1.5.0 and

[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] 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] [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] 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

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

2017-07-11 Thread Grant Henke (Code Review)
Hello 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 (#4). Change subject: KUDU-2066. Add experimental Gradle build support .. KUDU-2066.

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

2017-07-17 Thread Grant Henke (Code Review)
Hello 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 (#6). Change subject: KUDU-2066. Add experimental Gradle build support .. KUDU-2066.

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

2017-07-17 Thread Grant Henke (Code Review)
Hello 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 (#7). Change subject: KUDU-2066. Add experimental Gradle build support .. KUDU-2066.

[kudu-CR] KUDU-2067. Enable cfile checksumming by default

2017-07-12 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2067. Enable cfile checksumming by default .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7409/1//COMMIT_MSG Commit Message: Line 11:

[kudu-CR] KUDU-2067. Enable cfile checksumming by default

2017-07-12 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7409 to look at the new patch set (#2). Change subject: KUDU-2067. Enable cfile checksumming by default .. KUDU-2067.

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

2017-07-21 Thread Grant Henke (Code Review)
Hello Dan Burkert, 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 (#10). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-07-21 Thread Grant Henke (Code Review)
Hello Dan Burkert, 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 (#9). Change subject: KUDU-2066. Add experimental Gradle build support ..

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

2017-07-21 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7258/7/java/gradle/wrapper/gradle-wrapper.jar

[kudu-CR] Fix bug KUDU-2058

2017-07-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Fix bug KUDU-2058 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7491/1/java/kudu-client/src/main/java/org/apache/kudu/client/LocatedTablet.java

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

2017-07-24 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 (#13). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-07-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/7258/11/java/gradle/wrapper.gradle File

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

2017-07-24 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 (#12). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-07-24 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 (#14). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-07-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 13: Pushed a fix to unblock Todds iterative testing. Will follow up for a fix to the multibuild

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

2017-07-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 10: (3 comments) http://gerrit.cloudera.org:8080/#/c/7258/10/java/gradle/wrapper.gradle File

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

2017-07-24 Thread Grant Henke (Code Review)
Hello Dan Burkert, 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 (#11). Change subject: KUDU-2066. Add experimental Gradle build support

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

2017-07-20 Thread Grant Henke (Code Review)
Hello Dan Burkert, 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 (#8). Change subject: KUDU-2066. Add experimental Gradle build support ..

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

2017-07-20 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 7: (7 comments) http://gerrit.cloudera.org:8080/#/c/7258/7/java/gradle/shadow.gradle File

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

2017-07-12 Thread Grant Henke (Code Review)
Hello 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 (#5). Change subject: KUDU-2066. Add experimental Gradle build support .. KUDU-2066.

[kudu-CR] KUDU-2067. Enable cfile checksumming by default

2017-07-12 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7409 Change subject: KUDU-2067. Enable cfile checksumming by default .. KUDU-2067. Enable cfile checksumming by default Cfile checksums were

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

2017-07-12 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: KUDU-2066. Add experimental Gradle build support .. Patch Set 5: (1 comment)

[kudu-CR] [java] excluded slf4j from async dependency

2017-06-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7320 Change subject: [java] excluded slf4j from async dependency .. [java] excluded slf4j from async dependency The slf4j-api dependency

[kudu-CR] WIP: Add gradle build

2017-06-29 Thread Grant Henke (Code Review)
Hello 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 (#3). Change subject: WIP: Add gradle build .. WIP: Add gradle build Adds an

[kudu-CR] WIP: Add gradle build

2017-06-29 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: Add gradle build .. Patch Set 3: No need to review yet. Just pushing a rebase so Todd can test. -- To view, visit http://gerrit.cloudera.org:8080/7258 To

[kudu-CR] [java] excluded slf4j from async dependency

2017-06-29 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java] excluded slf4j from async dependency .. Patch Set 2: I don't have any formal write up on version ranges advice. But essentially the advice is that though

[kudu-CR] [java] excluded slf4j from async dependency

2017-06-29 Thread Grant Henke (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7320 to look at the new patch set (#2). Change subject: [java] excluded slf4j from async dependency ..

[kudu-CR](branch-1.2.x) [java-client] update protoc maven plugin

2017-07-05 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: [java-client] update protoc maven plugin .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7345 To unsubscribe, visit

  1   2   3   4   5   6   7   8   9   10   >