[kudu-CR] Add protobuf generated files to Eclipse project classpath. With this change Eclipse can compile all Java projects and allows to can run JUnit tests from Eclipse

2018-08-23 Thread Greg Solovyev (Code Review)
Hello Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11307 to review the following change. Change subject: Add protobuf generated files to Eclipse project classpath. With this change Eclipse can compile all Java projects and allows to can run

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-23 Thread Greg Solovyev (Code Review)
Greg Solovyev has abandoned this change. ( http://gerrit.cloudera.org:8080/11312 ) Change subject: Add protobuf generated files to Eclipse project classpath. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/11312

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-23 Thread Greg Solovyev (Code Review)
Hello Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11310 to review the following change. Change subject: Add protobuf generated files to Eclipse project classpath. .. Add

[kudu-CR] Add protobuf generated files to Eclipse project classpath. With this change Eclipse can compile all Java projects and allows to can run JUnit tests from Eclipse

2018-08-23 Thread Greg Solovyev (Code Review)
Greg Solovyev has abandoned this change. ( http://gerrit.cloudera.org:8080/11307 ) Change subject: Add protobuf generated files to Eclipse project classpath. With this change Eclipse can compile all Java projects and allows to can run JUnit tests from Eclipse

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-23 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11310 to look at the new patch set (#2). Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-23 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11310 to look at the new patch set (#3). Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-23 Thread Greg Solovyev (Code Review)
Hello Adar Dembo, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11312 to review the following change. Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-24 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11310 to look at the new patch set (#4). Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-24 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11310 to look at the new patch set (#5). Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 2: (1 comment)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 4: (1 comment)

[kudu-CR] Add protobuf generated files to Eclipse project classpath.

2018-08-27 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11310 to look at the new patch set (#6). Change subject: Add protobuf generated files to Eclipse project classpath.

[kudu-CR] [examples] Add basic Spark example (scala)

2018-10-29 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example (scala) .. Patch Set 8: (1 comment)

[kudu-CR] [examples] Add basic Spark example (scala)

2018-10-31 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example (scala) .. Patch Set 10: (6 comments)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-26 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 4: (1 comment)

[kudu-CR] [examples] Add basic Spark example written in Scala

2018-10-29 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example written in Scala .. Patch Set 8: Code-Review+1 -- To view, visit

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 3: It would be good to provide a working code sample

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12240 ) Change subject: KUDU-2456 Limit number of pending transactions logged .. Patch Set 1: (1 comment)

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-25 Thread Greg Solovyev (Code Review)
Greg Solovyev has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12847 Change subject: [KUDU-1344] Part 1: cmake install for executables .. [KUDU-1344] Part 1: cmake install for executables This change

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-25 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12847 ) Change subject: [KUDU-1344] Part 1: cmake install for executables .. Patch Set 1: (1 comment)

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12847 ) Change subject: [KUDU-1344] Part 1: cmake install for executables .. Patch Set 6: (1 comment)

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12847 to look at the new patch set (#6). Change subject: [KUDU-1344] Part 1: cmake install for executables

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12847 to look at the new patch set (#3). Change subject: [KUDU-1344] Part 1: cmake install for executables

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12847 to look at the new patch set (#4). Change subject: [KUDU-1344] Part 1: cmake install for executables

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12847 to look at the new patch set (#5). Change subject: [KUDU-1344] Part 1: cmake install for executables

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-27 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12847 ) Change subject: [KUDU-1344] Part 1: cmake install for executables .. Patch Set 2: (11 comments)

[kudu-CR] add document for KUDU-2080

2019-03-25 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12774 ) Change subject: add document for KUDU-2080 .. Patch Set 1: Code-Review+1 +1 (please address Adar's comment on wording) -- To view,

[kudu-CR] [KUDU-1344] Part 1: cmake install for executables

2019-03-26 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12847 to look at the new patch set (#2). Change subject: [KUDU-1344] Part 1: cmake install for executables

[kudu-CR] [master] introduce TTL-based cache

2019-03-04 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12652 ) Change subject: [master] introduce TTL-based cache .. Patch Set 1: (2 comments)

[kudu-CR] [master] introduce TTL-based cache

2019-03-04 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12652 ) Change subject: [master] introduce TTL-based cache .. Patch Set 1: (1 comment)

[kudu-CR] [java] Add private diff scan support

2019-03-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12689 ) Change subject: [java] Add private diff scan support .. Patch Set 5: (2 comments)

[kudu-CR] [java] Add private diff scan support

2019-03-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12689 ) Change subject: [java] Add private diff scan support .. Patch Set 6: (1 comment)

[kudu-CR] WIP [master] introduced SentryAuthzCache

2019-03-18 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12653 ) Change subject: WIP [master] introduced SentryAuthzCache .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.9.x) [master] introduce cache for location mapping assignments

2019-03-19 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12783 ) Change subject: [master] introduce cache for location mapping assignments .. Patch Set 1: (1 comment)

[kudu-CR](gh-pages) Add JVM testing blog

2019-03-19 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12795 ) Change subject: Add JVM testing blog .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2744: Expose RPC interface for diff scan

2019-03-19 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12592 ) Change subject: KUDU-2744: Expose RPC interface for diff scan .. Patch Set 4: (1 comment)

[kudu-CR](branch-1.9.x) [master] introduce cache for location mapping assignments

2019-03-20 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12783 ) Change subject: [master] introduce cache for location mapping assignments .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR](gh-pages) Add JVM testing blog

2019-03-20 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12795 ) Change subject: Add JVM testing blog .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] KUDU-2744: Expose RPC interface for diff scan

2019-03-20 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12592 ) Change subject: KUDU-2744: Expose RPC interface for diff scan .. Patch Set 6: (1 comment)

[kudu-CR](gh-pages) KUDU-2746 use /dyn/ in the URLs

2019-03-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12766 ) Change subject: KUDU-2746 use /dyn/ in the URLs .. Patch Set 1: Code-Review+1 LGTM. I tested the new links. -- To view, visit

[kudu-CR] [thirdparty] Bump RapidJSON version to 1.1.0

2019-03-07 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12643 ) Change subject: [thirdparty] Bump RapidJSON version to 1.1.0 .. Patch Set 9: Code-Review+1 I ran the build and ctest on Ubuntu18. All

[kudu-CR] KUDU-2099: drop Java 7 support

2019-03-21 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12816 ) Change subject: KUDU-2099: drop Java 7 support .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/12816

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#3). Change subject: KUDU-1900: add loopback check and test ..

[kudu-CR] docs: fix two broken links in the installation doc

2019-02-13 Thread Greg Solovyev (Code Review)
Greg Solovyev has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12480 Change subject: docs: fix two broken links in the installation doc .. docs: fix two broken links in the installation doc Change-Id:

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#5). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#4). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#2). Change subject: KUDU-1900: add loopback check and test .. KUDU-1900: add

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-13 Thread Greg Solovyev (Code Review)
Greg Solovyev has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12474 Change subject: KUDU-1900: add loopback check and test .. KUDU-1900: add loopback check and test This change modifies

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 6: (11 comments)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 6: (3 comments)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 6: (2 comments)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 9: Code-Review-1 I am going to upload one more change eliminating

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#11). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#10). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 11: I also ran security-itest on my mac with wifi turned off and

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#12). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#8). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#7). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#9). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#13). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-15 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 10: (4 comments)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12490 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 1: pardon, messed up Change-Id and created a separate review --

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12490 Change subject: KUDU-1900: add loopback check and test .. KUDU-1900: add loopback check and test This change modifies

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#6). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-14 Thread Greg Solovyev (Code Review)
Greg Solovyev has abandoned this change. ( http://gerrit.cloudera.org:8080/12490 ) Change subject: KUDU-1900: add loopback check and test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/12490 To unsubscribe,

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-19 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#15). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-19 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 10: (4 comments)

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-19 Thread Greg Solovyev (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12474 to look at the new patch set (#14). Change subject: KUDU-1900: add loopback check and test

[kudu-CR] KUDU-1900: add loopback check and test

2019-02-19 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12474 ) Change subject: KUDU-1900: add loopback check and test .. Patch Set 10: (2 comments)

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-29 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 7: Code-Review+1 -- To view, visit

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-28 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 5: What about a link to a sample script or an inline

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-30 Thread Greg Solovyev (Code Review)
Greg Solovyev has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12316 Change subject: KUDU-2477: highlight primary key in master Web UI .. KUDU-2477: highlight primary key in master Web UI This change

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-31 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12316 ) Change subject: KUDU-2477: highlight primary key in master Web UI .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-31 Thread Greg Solovyev (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12316 to look at the new patch set (#4). Change subject: KUDU-2477: highlight primary key in master Web UI

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-31 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12316 to look at the new patch set (#3). Change subject: KUDU-2477: highlight primary key in master Web UI .. KUDU-2477:

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-31 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12316 ) Change subject: KUDU-2477: highlight primary key in master Web UI .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2477: highlight primary key in master Web UI

2019-01-30 Thread Greg Solovyev (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12316 to look at the new patch set (#2). Change subject: KUDU-2477: highlight primary key in master Web UI .. KUDU-2477: