kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: [master] --location_mapping_cmd is experimental
Date Thu, 11 Oct 2018 02:01:54 GMT
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 <wdberkeley@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 <aserbin@cloudera.com>
Authored: Wed Oct 10 18:09:28 2018 -0700
Committer: Will Berkeley <wdberkeley@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(-)

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 "

View raw message