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 03:27:54 GMT
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(-)

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