[kudu-CR] itest: allow use of verifiers with EMCs that specify non-default block managers

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8208 ) Change subject: itest: allow use of verifiers with EMCs that specify non-default block managers .. Patch Set 4: (4 comments)

[kudu-CR](branch-1.4.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8224 Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. KUDU-2167: fix C++ client crash due to bad

[kudu-CR](branch-1.5.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8223 Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. KUDU-2167: fix C++ client crash due to bad

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. KUDU-2167: fix C++ client crash due

[kudu-CR] tool: new action for running mini-clusters

2017-10-05 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7853 to look at the new patch set (#12). Change subject: tool: new action for running mini-clusters

[kudu-CR] KUDU-1125: issue one catalog write per tablet report

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8090 ) Change subject: KUDU-1125: issue one catalog write per tablet report .. Patch Set 8: Verified+1 Overriding Jenkins, a few TSAN tests

[kudu-CR] KUDU-1125: issue one catalog write per tablet report

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8090 ) Change subject: KUDU-1125: issue one catalog write per tablet report .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#7). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] KUDU-2173: Partitions are pruned incorrectly when range-partitioned on a PK prefix

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8222 ) Change subject: KUDU-2173: Partitions are pruned incorrectly when range-partitioned on a PK prefix .. Patch Set 1: (1 comment) need to

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 3: Fixed a few grammar nits, otherwise pretty much unchanged.

[kudu-CR] itest: allow use of verifiers with EMCs that specify non-default block managers

2017-10-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8208 ) Change subject: itest: allow use of verifiers with EMCs that specify non-default block managers .. Patch Set 3: (10 comments)

[kudu-CR] itest: allow use of verifiers with EMCs that specify non-default block managers

2017-10-05 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8208 to look at the new patch set (#4). Change subject: itest: allow use of verifiers with EMCs that specify non-default block managers

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8220 to look at the new patch set (#3). Change subject: docs: clarify directory configs and usage .. docs: clarify

[kudu-CR] KUDU-2173: Partitions are pruned incorrectly when range-partitioned on a PK prefix

2017-10-05 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8222 to review the following change. Change subject: KUDU-2173: Partitions are pruned incorrectly when range-partitioned on a PK prefix

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 2: > Patch Set 2: > > > Do you

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 2: > Do you mind running a quick

[kudu-CR] KUDU-1125: issue one catalog write per tablet report

2017-10-05 Thread Adar Dembo (Code Review)
Hello Mike Percy, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8090 to look at the new patch set (#8). Change subject: KUDU-1125: issue one catalog write per tablet report

[kudu-CR] KUDU-1125: issue one catalog write per tablet report

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8090 ) Change subject: KUDU-1125: issue one catalog write per tablet report .. Patch Set 8: (8 comments)

[kudu-CR] WIP [raft consensus-itest] fix flake in TestSlowLeader

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7887 ) Change subject: WIP [raft_consensus-itest] fix flake in TestSlowLeader .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 2: Code-Review+2 looks good. Do

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8204 to look at the new patch set (#2). Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

[kudu-CR] WIP [consensus] adding/removing NON VOTER members

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8138 ) Change subject: WIP [consensus] adding/removing NON_VOTER members .. Patch Set 7: (17 comments) Looks pretty good. I agree we don't

[kudu-CR] KUDU-1125: issue one catalog write per tablet report

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8090 ) Change subject: KUDU-1125: issue one catalog write per tablet report .. Patch Set 7: Code-Review+1 (4 comments) lgtm after you address

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM .. Patch Set 4: Checking the failed tests. I should post the fix soon. --

[kudu-CR] KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8144 ) Change subject: KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction .. Patch Set 6: (2 comments)

[kudu-CR] KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock

2017-10-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8219 ) Change subject: KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock .. Patch Set 1: Checking the failed tests. I should

[kudu-CR] KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8204 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 1: (2 comments)

[kudu-CR] tool: new action for running mini-clusters

2017-10-05 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7853 to look at the new patch set (#11). Change subject: tool: new action for running mini-clusters

[kudu-CR] java: replace bespoke minicluster implementation with control shell

2017-10-05 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8194 to look at the new patch set (#4). Change subject: java: replace bespoke minicluster implementation with control shell

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#6). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 5: Looks like a bunch of the edits in later revs

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#5). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 4: (1 comment)

[kudu-CR] [webui] Allow custom response codes and headers

2017-10-05 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8141 ) Change subject: [webui] Allow custom response codes and headers .. [webui] Allow custom response codes and headers Previously,

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8220 to look at the new patch set (#2). Change subject: docs: clarify directory configs and usage .. docs: clarify

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 2: (2 comments)

[kudu-CR] [webui] Allow custom response codes and headers

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8141 ) Change subject: [webui] Allow custom response codes and headers .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 4: (1 comment)

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 1: (2 comments) e

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 4: (3 comments)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#4). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] [catalog manager] introduce replica selector

2017-10-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8161 ) Change subject: [catalog manager] introduce replica selector .. Patch Set 3: > (1 comment) And another option is ReplicaMatchPolicy

[kudu-CR] [catalog manager] introduce replica selector

2017-10-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8161 ) Change subject: [catalog manager] introduce replica selector .. Patch Set 3: (1 comment)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 3: (1 comment)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 3: (1 comment)

[kudu-CR] WIP [consensus] adding/removing NON VOTER members

2017-10-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8138 ) Change subject: WIP [consensus] adding/removing NON_VOTER members .. Patch Set 7: > (8 comments) > > This is looking pretty good

[kudu-CR] WIP: add kudu perf tool for latencies

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8221 Change subject: WIP: add kudu perf tool for latencies .. WIP: add kudu perf tool for latencies not worth reviewing yet, just wanted to

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8220 ) Change subject: docs: clarify directory configs and usage .. Patch Set 1: Not super confident in the `--fs_wal_dir=` change, but I

[kudu-CR] docs: clarify directory configs and usage

2017-10-05 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8220 Change subject: docs: clarify directory configs and usage .. docs: clarify directory configs and usage Added a short blurb about what the

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 3: (7 comments)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#3). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8218 to look at the new patch set (#2). Change subject: Allow the SASL protocol service name to be configurable

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 1: (8 comments)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 1: (2 comments)

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8218 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 1: (6 comments)

[kudu-CR] KUDU-2044 Tombstoned tablets show up in /metrics

2017-10-05 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7981 to look at the new patch set (#7). Change subject: KUDU-2044 Tombstoned tablets show up in /metrics

[kudu-CR] KUDU-2044 Tombstoned tablets show up in /metrics

2017-10-05 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/7981 ) Change subject: KUDU-2044 Tombstoned tablets show up in /metrics .. Patch Set 6: (6 comments)

[kudu-CR] KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries.

2017-10-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8037 ) Change subject: KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries. .. KUDU-1788. Increase Raft RPC

[kudu-CR] KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction

2017-10-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8144 ) Change subject: KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction .. Patch Set 6: (13 comments)

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM .. Patch Set 4: (18 comments)

[kudu-CR] KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock

2017-10-05 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8219 Change subject: KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock .. KUDU-2055 [part 4]: refactor BM to remove

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-05 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8162 to look at the new patch set (#4). Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM

[kudu-CR] KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction

2017-10-05 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8144 to look at the new patch set (#6). Change subject: KUDU-2055 [part 3]: Refactor BlockCreationTransaction and BlockDeletionTransaction

[kudu-CR] [tools] Add summary-only mode to ksck

2017-10-05 Thread Will Berkeley (Code Review)
Will Berkeley has abandoned this change. ( http://gerrit.cloudera.org:8080/7759 ) Change subject: [tools] Add summary-only mode to ksck .. Abandoned I'm going to do something on this eventually, but to do it right is a bigger

[kudu-CR] Allow the SASL protocol service name to be configurable

2017-10-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8218 Change subject: Allow the SASL protocol service name to be configurable .. Allow the SASL protocol service name to be configurable