Repository: kudu
Updated Branches:
refs/heads/branch-1.8.x d79f80eb0 -> bf8dbd3ef
[master] --location_mapping_cmd is experimental
Re-tagged --location_mapping_cmd: evolving --> experimental.
Not all pieces of the location-awareness is there yet, so let's
mark the location mapping flag as 'experimental' instead of 'evolving'.
The rationale is simple: the feature is not yet code-complete and
there might be some usability issues if using the location awareness
provisions at this point. E.g., location-unaware rebalancer would
violate the placement policies if run against a cluster with locations
defined, etc.
Change-Id: I783f07ff070d608deacfd6dd055fede80795b3ee
Reviewed-on: http://gerrit.cloudera.org:8080/11652
Tested-by: Kudu Jenkins
Reviewed-by: Will Berkeley <wdberkeley@gmail.com>
(cherry picked from commit b1102d757c35d409aed840a5fe0834ea9f7812a2)
Reviewed-on: http://gerrit.cloudera.org:8080/11653
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/bf8dbd3e
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/bf8dbd3e
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/bf8dbd3e
Branch: refs/heads/branch-1.8.x
Commit: bf8dbd3efef6b87bfd135e37028cf732a275461c
Parents: d79f80e
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Wed Oct 10 18:09:28 2018 -0700
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Thu Oct 11 03:26:14 2018 +0000
----------------------------------------------------------------------
src/kudu/master/ts_descriptor.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/bf8dbd3e/src/kudu/master/ts_descriptor.cc
----------------------------------------------------------------------
diff --git a/src/kudu/master/ts_descriptor.cc b/src/kudu/master/ts_descriptor.cc
index ae5398a..3a400b2 100644
--- a/src/kudu/master/ts_descriptor.cc
+++ b/src/kudu/master/ts_descriptor.cc
@@ -58,7 +58,7 @@ DEFINE_string(location_mapping_cmd, "",
"and consists of /-separated tokens each of which contains only "
"characters from the set [a-zA-Z0-9_-.]. If the cluster is not "
"using location awareness features this flag should not be set.");
-TAG_FLAG(location_mapping_cmd, evolving);
+TAG_FLAG(location_mapping_cmd, experimental);
DEFINE_bool(location_mapping_by_uuid, false,
"Whether the location command is given tablet server identifier "
|