[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8794 Change subject: kudu-hive: skip publishing artifact .. kudu-hive: skip publishing artifact kudu-hive is experimental, so we should avoid

[kudu-CR] KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client

2017-12-07 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8692 to look at the new patch set (#3). Change subject: KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: docs: updates to data dir behavior .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8778 To unsubscribe, visit

[kudu-CR] java: re-revisit maven shade configuration

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8792 ) Change subject: java: re-revisit maven shade configuration .. java: re-revisit maven shade configuration I introduced a bug in

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8778 To

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 4: Code-Review+1 (1 comment) just one moretypo nit

[kudu-CR] [quorum util] update criteria for non-voter replica eviction

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8679 ) Change subject: [quorum_util] update criteria for non-voter replica eviction .. Patch Set 3: (17 comments)

[kudu-CR] KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

2017-12-07 Thread Dan Burkert (Code Review)
Hello Michael Ho, Tidy Bot, Alexey Serbin, Kudu Jenkins, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8755 to look at the new patch set (#6). Change subject: KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8778 to look at the new patch set (#4). Change subject: docs: updates to data dir behavior ..

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 3: (11 comments) Yeah, I think the changes make sense. Also going with

[kudu-CR] KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8755 ) Change subject: KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional .. Patch Set 6: (7 comments)

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 5: (1 comment)

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8778 to look at the new patch set (#5). Change subject: docs: updates to data dir behavior ..

[kudu-CR](branch-1.6.x) java: re-revisit maven shade configuration

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8793 Change subject: java: re-revisit maven shade configuration .. java: re-revisit maven shade configuration I introduced a bug in the

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8789 ) Change subject: KUDU-2228: Make Messenger options configurable .. Patch Set 3: (7 comments)

[kudu-CR] java: re-revisit maven shade configuration

2017-12-07 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8792 to review the following change. Change subject: java: re-revisit maven shade configuration ..

[kudu-CR] java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8792 ) Change subject: java: re-revisit maven shade configuration .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Add initial internal INT128/ int128 support

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 17: (1 comment) Nice, these literals are way-cool.

[kudu-CR](branch-1.6.x) java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8793 ) Change subject: java: re-revisit maven shade configuration .. java: re-revisit maven shade configuration I introduced a

[kudu-CR](branch-1.6.x) java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8793 ) Change subject: java: re-revisit maven shade configuration .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. docs: updates to data dir behavior Kudu tservers are now able to survive

[kudu-CR](gh-pages) Updating web site for Kudu 1.6.0 release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8799 ) Change subject: Updating web site for Kudu 1.6.0 release .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR] [quorum util] update criteria for non-voter replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8679 ) Change subject: [quorum_util] update criteria for non-voter replica eviction .. Patch Set 3: (17 comments)

[kudu-CR](gh-pages) Updating web site for Kudu 1.6.0 release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8799 Change subject: Updating web site for Kudu 1.6.0 release .. Updating web site for Kudu 1.6.0 release Change-Id:

[kudu-CR] docs: Add scripts to automate web site update for releases

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8800 Change subject: docs: Add scripts to automate web site update for releases .. docs: Add scripts to automate web site update for releases

[kudu-CR](gh-pages) Make it possible to automate the docs release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8798 Change subject: Make it possible to automate the docs release .. Make it possible to automate the docs release Change-Id:

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8778

[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8794 ) Change subject: kudu-hive: skip publishing artifact .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8794

[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8794 ) Change subject: kudu-hive: skip publishing artifact .. kudu-hive: skip publishing artifact kudu-hive is experimental, so

[kudu-CR](branch-1.6.x) docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8797 ) Change subject: docs: updates to data dir behavior .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8797 ) Change subject: docs: updates to data dir behavior .. docs: updates to data dir behavior Kudu tservers are now able to survive

[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8794 ) Change subject: kudu-hive: skip publishing artifact .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8797 Change subject: docs: updates to data dir behavior .. docs: updates to data dir behavior Kudu tservers are now able to survive select disk

[kudu-CR] KUDU-1097: more robust behavior for 'replica move'

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/8786 ) Change subject: KUDU-1097: more robust behavior for 'replica move' .. Abandoned It's a part of https://gerrit.cloudera.org/#/c/8679/ now -- To

[kudu-CR](gh-pages) Updating web site for Kudu 1.6.0 release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8799 ) Change subject: Updating web site for Kudu 1.6.0 release .. Updating web site for Kudu 1.6.0 release Change-Id:

[kudu-CR](gh-pages) Make it possible to automate the docs release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8798 ) Change subject: Make it possible to automate the docs release .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Make it possible to automate the docs release

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8798 ) Change subject: Make it possible to automate the docs release .. Make it possible to automate the docs release Change-Id:

[kudu-CR] KUDU-1097: more robust criteria for replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8679 ) Change subject: KUDU-1097: more robust criteria for replica eviction .. Patch Set 6: Verified+1 unrelated flake in tablet_copy-itest

[kudu-CR] KUDU-1097: more robust criteria for replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8679 ) Change subject: KUDU-1097: more robust criteria for replica eviction .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] [quorum util] update criteria for non-voter replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8679 to look at the new patch set (#4). Change subject: [quorum_util] update criteria for non-voter replica eviction

[kudu-CR] [quorum util] update criteria for voter replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/8776 ) Change subject: [quorum_util] update criteria for voter replica eviction .. Abandoned Now this is a part of:

[kudu-CR] KUDU-1097: more robust criteria for replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8679 to look at the new patch set (#5). Change subject: KUDU-1097: more robust criteria for replica eviction

[kudu-CR] KUDU-1097: more robust criteria for replica eviction

2017-12-07 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8679 to look at the new patch set (#6). Change subject: KUDU-1097: more robust criteria for replica eviction

[kudu-CR] KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

2017-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8755 ) Change subject: KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional .. KUDU-2121: fix SASL PLAIN fallback with

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8789 ) Change subject: KUDU-2228: Make Messenger options configurable .. Patch Set 2: (13 comments)

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Sailesh Mukil (Code Review)
Hello Michael Ho, Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8789 to look at the new patch set (#2). Change subject: KUDU-2228: Make Messenger options configurable

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8789 ) Change subject: KUDU-2228: Make Messenger options configurable .. Patch Set 3: The last failure looks like a flaky test in:

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Sailesh Mukil (Code Review)
Hello Michael Ho, Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8789 to look at the new patch set (#3). Change subject: KUDU-2228: Make Messenger options configurable

[kudu-CR] KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

2017-12-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8755 ) Change subject: KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional .. Patch Set 5: (3 comments) Sorry for the late

[kudu-CR] [rpc] WIP: Introduce configurable options to Messenger

2017-12-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has abandoned this change. ( http://gerrit.cloudera.org:8080/6520 ) Change subject: [rpc] WIP: Introduce configurable options to Messenger .. Abandoned Re-did the patch here:

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8789 Change subject: KUDU-2228: Make Messenger options configurable .. KUDU-2228: Make Messenger options configurable Currently, the RPC

[kudu-CR] KUDU-2228: Make Messenger options configurable

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8789 ) Change subject: KUDU-2228: Make Messenger options configurable .. Patch Set 3: (1 comment)

[kudu-CR] [cluster itest util] update WaitForReplicasReportedToMaster

2017-12-07 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8671 to look at the new patch set (#2). Change subject: [cluster_itest_util] update WaitForReplicasReportedToMaster

[kudu-CR] KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional

2017-12-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8755 ) Change subject: KUDU-2121: fix SASL PLAIN fallback with rpc-authentication=optional .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] docs: updates to data dir behavior

2017-12-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8778 ) Change subject: docs: updates to data dir behavior .. Patch Set 3: (11 comments) looks good. added a few suggestions / thoughts, lmk