[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 8: Code-Review+1 (3 comments)

[kudu-CR] [clock] change clock source selection for 'auto'

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17582 ) Change subject: [clock] change clock source selection for 'auto' .. Patch Set 3: (1 comment)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#8). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#7). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 6: (2 comments)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#6). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [clock] change clock source selection for 'auto'

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17582 ) Change subject: [clock] change clock source selection for 'auto' .. Patch Set 2: (2 comments)

[kudu-CR] [clock] change clock source selection for 'auto'

2021-06-11 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17582 to look at the new patch set (#3). Change subject: [clock] change clock source selection for 'auto'

[kudu-CR] [clock] use GTEST SKIP in BuiltinNtpWithMiniChronydTest

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17587 ) Change subject: [clock] use GTEST_SKIP in BuiltinNtpWithMiniChronydTest .. [clock] use GTEST_SKIP in

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 3: Verified+1 (1 comment)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: [util] flag to specify hosts to fail DNS resolution .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17584 To

[kudu-CR] [clock] use GTEST SKIP in BuiltinNtpWithMiniChronydTest

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17587 ) Change subject: [clock] use GTEST_SKIP in BuiltinNtpWithMiniChronydTest .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [clock] use GTEST SKIP in BuiltinNtpWithMiniChronydTest

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17587 Change subject: [clock] use GTEST_SKIP in BuiltinNtpWithMiniChronydTest .. [clock] use GTEST_SKIP in BuiltinNtpWithMiniChronydTest

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 5: Code-Review+1 (2 comments)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 5: (3 comments)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17584 to look at the new patch set (#3). Change subject: [util] flag to specify hosts to fail DNS resolution

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#5). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 4: (2 comments)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 4: (2 comments)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#4). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 3: (1 comment)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 2: (1 comment)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17584 to look at the new patch set (#2). Change subject: [util] flag to specify hosts to fail DNS resolution

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#3). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 2: (2 comments)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 1: (1 comment)

[kudu-CR] [rest] add oat++ framework to kudu

2021-06-11 Thread Khazar Mammadli (Code Review)
Hello Attila Bukor, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17554 to look at the new patch set (#6). Change subject: [rest] add oat++ framework to kudu .. [rest] add

[kudu-CR] WIP [clock] change clock source selection for 'auto'

2021-06-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17582 ) Change subject: WIP [clock] change clock source selection for 'auto' .. Patch Set 2: (1 comment) The overall change/approach makes

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 2: Verified+1 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17585 to look at the new patch set (#2). Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17585 Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. [consensus] KUDU-2302: don't crash if new leader

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17584 Change subject: [util] flag to specify hosts to fail DNS resolution .. [util] flag to specify hosts to fail DNS resolution This patch

[kudu-CR] wip

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/17581 ) Change subject: wip .. Abandoned submitted by mistake before squashing commits -- To view, visit http://gerrit.cloudera.org:8080/17581 To

[kudu-CR] WIP [clock] change clock source selection for 'auto'

2021-06-11 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17582 to look at the new patch set (#2). Change subject: WIP [clock] change clock source selection for 'auto' .. WIP

[kudu-CR] WIP [clock] change clock source selection for 'auto'

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17582 Change subject: WIP [clock] change clock source selection for 'auto' .. WIP [clock] change clock source selection for 'auto' This

[kudu-CR] wip

2021-06-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17581 Change subject: wip .. wip Change-Id: Ib5e9c4cda6318a56252678447191ea8f265fb9fd --- M src/kudu/clock/builtin_ntp.cc M