[kudu-CR] [docs] Add changing master hostnames workflow

2018-08-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11058 ) Change subject: [docs] Add changing master hostnames workflow .. Patch Set 3: (1 comment)

[kudu-CR] [location awareness] Assign locations to registering tablet servers

2018-08-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/5 ) Change subject: [location_awareness] Assign locations to registering tablet servers .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] WIP: Move default sanitizer options into build from shell scripts

2018-08-09 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11176 to look at the new patch set (#5). Change subject: WIP: Move default sanitizer options into build from shell scripts

[kudu-CR] WIP: Move default sanitizer options into build from shell scripts

2018-08-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: WIP: Move default sanitizer options into build from shell scripts .. Patch Set 4: (3 comments)

[kudu-CR] WIP: Move default sanitizer options into build from shell scripts

2018-08-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: WIP: Move default sanitizer options into build from shell scripts .. Patch Set 4: (3 comments)

[kudu-CR] WIP: Move default sanitizer options into build from shell scripts

2018-08-09 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11176 to look at the new patch set (#4). Change subject: WIP: Move default sanitizer options into build from shell scripts

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: Enable TSAN and LSAN supressions in Java dist-tests .. Patch Set 2: > I think there is actually some way to link in

[kudu-CR] [location awareness] Assign locations to registering tablet servers

2018-08-09 Thread Will Berkeley (Code Review)
Hello Fengling Wang, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5 to look at the new patch set (#5). Change subject: [location_awareness] Assign locations to registering tablet servers

[kudu-CR] [location awareness] Assign locations to registering tablet servers

2018-08-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/5 ) Change subject: [location_awareness] Assign locations to registering tablet servers .. Patch Set 4: (5 comments)

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: Enable TSAN and LSAN supressions in Java dist-tests .. Patch Set 2: run-test gets called by ctest. See CMakeLists.txt

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: Enable TSAN and LSAN supressions in Java dist-tests .. Patch Set 2: I was looking for where run-test.sh is called

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 ) Change subject: Enable TSAN and LSAN supressions in Java dist-tests .. Patch Set 2: Doesn't this break non-distributed test runs? I

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 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/11176 to look at the new patch set (#2). Change subject: Enable TSAN and LSAN supressions in Java dist-tests

[kudu-CR] Enable TSAN and LSAN supressions in Java dist-tests

2018-08-09 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11176 Change subject: Enable TSAN and LSAN supressions in Java dist-tests .. Enable TSAN and LSAN supressions in Java dist-tests Moves the