Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 6F8A8200CC9 for ; Mon, 17 Jul 2017 10:18:06 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 6E35A163AAF; Mon, 17 Jul 2017 08:18:06 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id BCCEB163A2F for ; Mon, 17 Jul 2017 10:18:05 +0200 (CEST) Received: (qmail 89903 invoked by uid 500); 17 Jul 2017 08:18:04 -0000 Mailing-List: contact issues-help@ambari.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ambari.apache.org Delivered-To: mailing list issues@ambari.apache.org Received: (qmail 89894 invoked by uid 99); 17 Jul 2017 08:18:04 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 17 Jul 2017 08:18:04 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 32C1A18088E for ; Mon, 17 Jul 2017 08:18:04 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -100.002 X-Spam-Level: X-Spam-Status: No, score=-100.002 tagged_above=-999 required=6.31 tests=[RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id V_99x0aTqpVg for ; Mon, 17 Jul 2017 08:18:03 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTP id 3ED1C60D92 for ; Mon, 17 Jul 2017 08:18:03 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id A5EEAE08A0 for ; Mon, 17 Jul 2017 08:18:02 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 3821324766 for ; Mon, 17 Jul 2017 08:18:01 +0000 (UTC) Date: Mon, 17 Jul 2017 08:18:01 +0000 (UTC) From: "Attila Magyar (JIRA)" To: issues@ambari.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (AMBARI-21442) Ambari updates memory settings in blueprint incorrectly MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Mon, 17 Jul 2017 08:18:06 -0000 [ https://issues.apache.org/jira/browse/AMBARI-21442?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Attila Magyar updated AMBARI-21442: ----------------------------------- Attachment: AMBARI-21442.patch > Ambari updates memory settings in blueprint incorrectly > ------------------------------------------------------- > > Key: AMBARI-21442 > URL: https://issues.apache.org/jira/browse/AMBARI-21442 > Project: Ambari > Issue Type: Bug > Components: ambari-server > Affects Versions: 2.2.2 > Reporter: Attila Magyar > Assignee: Attila Magyar > Fix For: 3.0.0 > > Attachments: AMBARI-21442.patch > > > Ambari appended a 'm' suffix to certain memory settings related properties during blueprint install. This was implemented in the MPropertyUpdater. > For example if namenode_heapsize was 512 then Ambari updated it to 512m. However if the property already had a suffix like 4g then it was updated to be 4gm. > This patch does 2 things differently > 1. Instead of the hardcoded 'm' suffix, it uses the unit that is defined in the stack (if the stack doesn't define anything it falls back using 'm' as before). > 2. It checks if the property already has some unit, and if that unit doesn't match the stack defined unit, then the blueprint will be rejected with an error (this case has never worked before) > For examples: > 1. 4g is rejected if the stack defined unit is MB. > 2. 4 becomes 4m if the stack defined unit is MB. > 3. 4m stays 4m if the stack defined unit is MB. > MPropertyUpdated was replaced with UnitUpdater. But a new TopologyValidator was also introduced. The purpose of this is to catch invalid properties earlier. -- This message was sent by Atlassian JIRA (v6.4.14#64029)