[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 8: Created KUDU-3122 to track the 'disable

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 7: (1 comment)

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#8). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 7: (1 comment)

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 7: Failure appears to be flaky test via ASAN.

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 6: (2 comments) New patch set. Having a hard

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#7). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#6). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 5: These test failures are not in the Java code,

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#4). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#3). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 1: (6 comments) New patch set. Thanks for the

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Hello Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15882 to look at the new patch set (#2). Change subject: [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15882 ) Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. Patch Set 1: (3 comments) First draft of the row count

[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15882 Change subject: [KUDU-3116] Enhance KuduContext row operation metrics .. [KUDU-3116] Enhance KuduContext row operation metrics

[kudu-CR] KUDU-3085: Create Scala/SBT integration test example.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15535 ) Change subject: KUDU-3085: Create Scala/SBT integration test example. .. Patch Set 4: > Patch Set 3: > > I still see them. Maybe a

[kudu-CR] KUDU-3085: Create Scala/SBT integration test example.

Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15535 to look at the new patch set (#4). Change subject: KUDU-3085: Create Scala/SBT integration test example.

[kudu-CR] KUDU-3085: Create Scala/SBT integration test example.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15535 ) Change subject: KUDU-3085: Create Scala/SBT integration test example. .. Patch Set 3: > Patch Set 2: > > (1 comment) > > Looks good.

[kudu-CR] KUDU-3085: Create Scala/SBT integration test example.

Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15535 to look at the new patch set (#3). Change subject: KUDU-3085: Create Scala/SBT integration test example.

[kudu-CR] KUDU-3085: Create Scala/SBT integration test example.

Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15535 to look at the new patch set (#2). Change subject: KUDU-3085: Create Scala/SBT integration test example.

[kudu-CR] [KUDU-3085] Create Scala/SBT integration test example.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/15535 ) Change subject: [KUDU-3085] Create Scala/SBT integration test example. .. Patch Set 1: (7 comments) Thanks for the review!

[kudu-CR] [KUDU-3085] Create Scala/SBT integration test example.

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15535 Change subject: [KUDU-3085] Create Scala/SBT integration test example. .. [KUDU-3085] Create Scala/SBT integration test example. This

[kudu-CR] [java] Add useBinJar Gradle property

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/13195 ) Change subject: [java] Add useBinJar Gradle property .. Patch Set 3: Code-Review+1 (1 comment) I like it. We've also found a

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#11). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#10). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#9). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 8: (2 comments) Added comments to the

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#8). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 7: Looks like a flaky test:

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#6). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12406 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2411: Remove binary test artifact files on JVM exit

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12377 ) Change subject: KUDU-2411: Remove binary test artifact files on JVM exit .. Patch Set 2: LGTM -- To view, visit

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 3: (3 comments) New changes are incoming.

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#4). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Altered binary jar extractor API

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12319 to look at the new patch set (#4). Change subject: KUDU-2411: Altered binary jar extractor API

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 2: (3 comments)

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12318 to look at the new patch set (#2). Change subject: KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Altered binary jar extractor API

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12319 to look at the new patch set (#3). Change subject: KUDU-2411: Altered binary jar extractor API

[kudu-CR] KUDU-2411: Altered binary jar extractor API

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12319 ) Change subject: KUDU-2411: Altered binary jar extractor API .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-2411: Altered binary jar extractor API

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12319 Change subject: KUDU-2411: Altered binary jar extractor API .. KUDU-2411: Altered binary jar extractor API The jar extractor API now

[kudu-CR] KUDU-2411: Altered binary jar extractor API

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12319 to look at the new patch set (#2). Change subject: KUDU-2411: Altered binary jar extractor API

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12318 Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. KUDU-2411: Example integration test using KuduMiniCluster

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 1: (1 comment) A simple example of wiring it

[kudu-CR] binary artifact: generate properties file

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12314 ) Change subject: binary artifact: generate properties file .. Patch Set 1: Code-Review+1 Looks good. -- To view, visit

[kudu-CR] [build] Fix mini cluster binary relocation on Mac OS

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12310 ) Change subject: [build] Fix mini cluster binary relocation on Mac OS .. Patch Set 1: (1 comment) Looks good, just a question about

[kudu-CR] [build] Fix mini cluster binary relocation on Mac OS

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12310 ) Change subject: [build] Fix mini cluster binary relocation on Mac OS .. Patch Set 1: Code-Review+1 Ship it. -- To view, visit

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12141 to look at the new patch set (#6). Change subject: KUDU-2411: Add OS/Arch detection to binary extract

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12141 ) Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. Patch Set 5: (3 comments) Updated with the specified changes.

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12141 ) Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. Patch Set 5: One new failure:

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12141 ) Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. Patch Set 4: Jenkins appears to fail on unrelated Python code:

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12141 to look at the new patch set (#4). Change subject: KUDU-2411: Add OS/Arch detection to binary extract

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12141 to look at the new patch set (#3). Change subject: KUDU-2411: Add OS/Arch detection to binary extract

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12141 to look at the new patch set (#2). Change subject: KUDU-2411: Add OS/Arch detection to binary extract

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12141 ) Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. Patch Set 1: (1 comment) I will cleanup these items and

[kudu-CR] java: KuduBinaryExtractor should use the thread context classloader

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12147 ) Change subject: java: KuduBinaryExtractor should use the thread context classloader .. Patch Set 1: Code-Review+1 LGTM -- To

[kudu-CR] java: KuduBinaryExtractor should use the thread context classloader

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12147 ) Change subject: java: KuduBinaryExtractor should use the thread context classloader .. Patch Set 1: I think this will work nicely,

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12141 ) Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. Patch Set 1: It is not fully tested yet. We still need to add

[kudu-CR] KUDU-2411: Add OS/Arch detection to binary extract

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12141 Change subject: KUDU-2411: Add OS/Arch detection to binary extract .. KUDU-2411: Add OS/Arch detection to binary extract OS/Arch

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12104 ) Change subject: KUDU-2411: Initial draft of kudu binary jar extractor. .. Patch Set 4: (1 comment) Will remove the usage of the

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12104 to look at the new patch set (#5). Change subject: KUDU-2411: Initial draft of kudu binary jar extractor.

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12104 to look at the new patch set (#4). Change subject: KUDU-2411: Initial draft of kudu binary jar extractor.

[kudu-CR] java: add log4j.properties file to kudu-test-utils module

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12040 ) Change subject: java: add log4j.properties file to kudu-test-utils module .. Patch Set 3: (1 comment) One minor comment on the

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12104 ) Change subject: KUDU-2411: Initial draft of kudu binary jar extractor. .. Patch Set 2: (5 comments) Thank you for the feedback!

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Hello Mike Percy, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12104 to look at the new patch set (#3). Change subject: KUDU-2411: Initial draft of kudu binary jar extractor.

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12104 to look at the new patch set (#2). Change subject: KUDU-2411: Initial draft of kudu binary jar extractor.

[kudu-CR] KUDU-2411: Initial draft of kudu binary jar extractor.

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12104 Change subject: KUDU-2411: Initial draft of kudu binary jar extractor. .. KUDU-2411: Initial draft of kudu binary jar extractor.

[kudu-CR] KUDU-2489: Improve runtime of slow java test.

Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/11365 ) Change subject: KUDU-2489: Improve runtime of slow java test. .. Patch Set 2: > > Patch Set 1: > > > > (1 comment) > > > > This

[kudu-CR] KUDU-2489: Improve runtime of slow java test.

Hello Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11365 to look at the new patch set (#2). Change subject: KUDU-2489: Improve runtime of slow java test.

[kudu-CR] Improve runtime of slow java client security test.

Brian McDevitt has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11365 Change subject: Improve runtime of slow java client security test. .. Improve runtime of slow java client security test. KUDU-2489 -