From commits-return-6502-archive-asf-public=cust-asf.ponee.io@kudu.apache.org Thu Oct 11 05:27:56 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 8943D180672 for ; Thu, 11 Oct 2018 05:27:55 +0200 (CEST) Received: (qmail 16189 invoked by uid 500); 11 Oct 2018 03:27:54 -0000 Mailing-List: contact commits-help@kudu.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@kudu.apache.org Delivered-To: mailing list commits@kudu.apache.org Received: (qmail 16179 invoked by uid 99); 11 Oct 2018 03:27:54 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Oct 2018 03:27:54 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 4EA52DFD43; Thu, 11 Oct 2018 03:27:54 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: alexey@apache.org To: commits@kudu.apache.org Message-Id: <1fb29c84cb864c8f9f8810399cdffeba@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: kudu git commit: [master] --location_mapping_cmd is experimental Date: Thu, 11 Oct 2018 03:27:54 +0000 (UTC) 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 (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 Authored: Wed Oct 10 18:09:28 2018 -0700 Committer: Alexey Serbin 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 "