[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 3: Verified+1 Unrelated flake in

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11909 ) Change subject: [backup] Ensure random tests are reproducible .. Patch Set 2: It uses a new instance of java.util.Random each time.

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11909 ) Change subject: [backup] Ensure random tests are reproducible .. Patch Set 2: Verified+1 -- To view, visit

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11911 to review the following change. Change subject: KUDU-2378: fix another aligned load crash

[kudu-CR] revert change to exactly once writes-itest

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has abandoned this change. ( http://gerrit.cloudera.org:8080/11836 ) Change subject: revert change to exactly_once_writes-itest .. Abandoned This "leak" was a false positive, see

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [backup] Ensure random tests are reproducible .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11909 To unsubscribe,

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11909 ) Change subject: [backup] Ensure random tests are reproducible .. [backup] Ensure random tests are reproducible Changes the use

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11909 ) Change subject: [backup] Ensure random tests are reproducible .. Patch Set 2: The TSAN build cause the create table master crash

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 2: (1 comment)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11905 to look at the new patch set (#3). Change subject: [dist-test] build environment for tests in loop mode

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11908 to look at the new patch set (#2). Change subject: gradle: expose testRandomSeed property .. gradle:

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. Patch Set 2: (1 comment)

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11909 to look at the new patch set (#2). Change subject: [backup] Ensure random tests are reproducible ..

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 4: (2 comments)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11681 ) Change subject: Add "network_plane" as part of ConnectionId .. Add "network_plane" as part of ConnectionId The motivation for

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. Patch Set 2: Verified+1 Overriding Jenkins, DefaultSourceTest failed three

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. gradle: expose testRandomSeed property In order to read a system

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11681 ) Change subject: Add "network_plane" as part of ConnectionId .. Patch Set 1: (1 comment)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 3: (4 comments)

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1

[kudu-CR] (01/05) delta store: avoid copying deleted row data in ApplyUpdates

2018-11-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11856 ) Change subject: (01/05) delta_store: avoid copying deleted row data in ApplyUpdates .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 3: (4 comments)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11905 to look at the new patch set (#4). Change subject: [dist-test] build environment for tests in loop mode

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11909 ) Change subject: [backup] Ensure random tests are reproducible .. Patch Set 2: Code-Review+2 So what PRNG instance do Random.nextFoo()

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. [dist-test] build environment for tests in loop mode

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 4: Verified+1 Unrelated flake in

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11907 to look at the new patch set (#2). Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables

[kudu-CR] [java] Upgrade to Spark 2.4

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11912 Change subject: [java] Upgrade to Spark 2.4 .. [java] Upgrade to Spark 2.4 In Spark 2.4 spark-avro is now a part of Spark itself. This

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 2: (1 comment)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11886 to look at the new patch set (#5). Change subject: run-test: don't report leaks that don't fail test

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11905 to look at the new patch set (#2). Change subject: [dist-test] build environment for tests in loop mode ..

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11911 to look at the new patch set (#3). Change subject: KUDU-2378: fix another aligned load crash

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. [cmake] no empty KUDU_DATA_FILES ctest env variables

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. Patch Set 2: (1 comment)

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. Patch Set 4: > (1 comment) Yes, if(DATA_FILES_LIST) works,

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11911 to look at the new patch set (#2). Change subject: KUDU-2378: fix another aligned load crash

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11907 to look at the new patch set (#3). Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11911 to look at the new patch set (#4). Change subject: KUDU-2378: fix another aligned load crash

[kudu-CR] (02/05) delta store: convert DeltaIterator::PrepareBatch flags into bitfield

2018-11-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11857 ) Change subject: (02/05) delta_store: convert DeltaIterator::PrepareBatch flags into bitfield .. Patch Set 2: (2 comments) nice little

[kudu-CR] [java] Upgrade to Spark 2.4

2018-11-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11912 ) Change subject: [java] Upgrade to Spark 2.4 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/11912 To

[kudu-CR] (01/05) delta store: avoid copying deleted row data in ApplyUpdates

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11856 ) Change subject: (01/05) delta_store: avoid copying deleted row data in ApplyUpdates .. (01/05) delta_store: avoid copying

[kudu-CR] (03/05) delta store: support iteration with snap to exclude

2018-11-08 Thread Adar Dembo (Code Review)
Hello Mike Percy, David Ribeiro Alves, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11858 to look at the new patch set (#3). Change subject: (03/05) delta_store: support iteration with snap_to_exclude

[kudu-CR] [compaction] KUDU-1400: Improve rowset compaction policy to consider merging small DRSs

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11869 ) Change subject: [compaction] KUDU-1400: Improve rowset compaction policy to consider merging small DRSs .. Patch Set 4: (2 comments)

[kudu-CR] [java] Upgrade to Spark 2.4

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11912 ) Change subject: [java] Upgrade to Spark 2.4 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/11912 To

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. KUDU-2378: fix another aligned load crash The

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] KUDU-2378: fix another aligned load crash

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11911 ) Change subject: KUDU-2378: fix another aligned load crash .. Patch Set 4: Verified+1 Overriding Jenkins, DefaultSourceTest failed in

[kudu-CR] (02/05) delta store: convert DeltaIterator::PrepareBatch flags into bitfield

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11857 ) Change subject: (02/05) delta_store: convert DeltaIterator::PrepareBatch flags into bitfield .. Patch Set 2: (3 comments)

[kudu-CR] (02/05) delta store: convert DeltaIterator::PrepareBatch flags into bitfield

2018-11-08 Thread Adar Dembo (Code Review)
Hello Mike Percy, David Ribeiro Alves, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11857 to look at the new patch set (#3). Change subject: (02/05) delta_store: convert DeltaIterator::PrepareBatch flags into bitfield

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11681 ) Change subject: Add "network_plane" as part of ConnectionId .. Patch Set 4: (1 comment)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 7: Turns out IWYU was unhappy too. -- To view, visit

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11907 to look at the new patch set (#4). Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Adar Dembo (Code Review)
Hello Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11908 to review the following change. Change subject: gradle: expose testRandomSeed property .. gradle: expose

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-08 Thread Michael Ho (Code Review)
Hello Kudu Jenkins, Adar Dembo, Sailesh Mukil, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11681 to look at the new patch set (#5). Change subject: Add "network_plane" as part of ConnectionId

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 1: (3 comments)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11681 ) Change subject: Add "network_plane" as part of ConnectionId .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] [cmake] don't add empty KUDU DATA FILES env variable

2018-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11907 Change subject: [cmake] don't add empty KUDU_DATA_FILES env variable .. [cmake] don't add empty KUDU_DATA_FILES env variable This

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 6: (1 comment)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11886 to look at the new patch set (#6). Change subject: run-test: don't report leaks that don't fail test

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. Patch Set 2: (1 comment)

[kudu-CR] gradle: expose testRandomSeed property

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11908 ) Change subject: gradle: expose testRandomSeed property .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2599: fix flaky DefaultSourceTest.testSocketReadTimeoutPropagation

2018-11-08 Thread Adar Dembo (Code Review)
Hello Mike Percy, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11916 to review the following change. Change subject: KUDU-2599: fix flaky DefaultSourceTest.testSocketReadTimeoutPropagation

[kudu-CR] KUDU-2599: fix flaky DefaultSourceTest.testSocketReadTimeoutPropagation

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11916 ) Change subject: KUDU-2599: fix flaky DefaultSourceTest.testSocketReadTimeoutPropagation .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] (03/05) delta store: support iteration with snap to exclude

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11858 ) Change subject: (03/05) delta_store: support iteration with snap_to_exclude .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] (03/05) delta store: support iteration with snap to exclude

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11858 ) Change subject: (03/05) delta_store: support iteration with snap_to_exclude .. Patch Set 3: Verified+1 Overriding Jenkins, the failure

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. run-test: don't report leaks that don't fail test

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 6: (1 comment)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 7: Code-Review+2 Carrying forward Adar's +2 -- To

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11886 ) Change subject: run-test: don't report leaks that don't fail test .. Patch Set 5: (1 comment)

[kudu-CR] run-test: don't report leaks that don't fail test

2018-11-08 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11886 to look at the new patch set (#7). Change subject: run-test: don't report leaks that don't fail test

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] [dist-test] build environment for tests in loop mode

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11905 ) Change subject: [dist-test] build environment for tests in loop mode .. Patch Set 2: (1 comment)

[kudu-CR] [backup] Ensure random tests are reproducible

2018-11-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11909 Change subject: [backup] Ensure random tests are reproducible .. [backup] Ensure random tests are reproducible Changes the use of Random

[kudu-CR] [cmake] no empty KUDU DATA FILES ctest env variables

2018-11-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11907 ) Change subject: [cmake] no empty KUDU_DATA_FILES ctest env variables .. Patch Set 4: Code-Review+2 -- To view, visit