You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2018/10/11 02:01:54 UTC
kudu git commit: [master] --location_mapping_cmd is experimental
Repository: kudu
Updated Branches:
refs/heads/master 61aed1a58 -> b1102d757
[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 <wd...@gmail.com>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/b1102d75
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/b1102d75
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/b1102d75
Branch: refs/heads/master
Commit: b1102d757c35d409aed840a5fe0834ea9f7812a2
Parents: 61aed1a
Author: Alexey Serbin <as...@cloudera.com>
Authored: Wed Oct 10 18:09:28 2018 -0700
Committer: Will Berkeley <wd...@gmail.com>
Committed: Thu Oct 11 01:54:01 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/b1102d75/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 "