Return-Path: X-Original-To: apmail-hadoop-yarn-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-yarn-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A816C17B48 for ; Wed, 25 Mar 2015 15:21:58 +0000 (UTC) Received: (qmail 7065 invoked by uid 500); 25 Mar 2015 15:21:53 -0000 Delivered-To: apmail-hadoop-yarn-issues-archive@hadoop.apache.org Received: (qmail 7014 invoked by uid 500); 25 Mar 2015 15:21:53 -0000 Mailing-List: contact yarn-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: yarn-issues@hadoop.apache.org Delivered-To: mailing list yarn-issues@hadoop.apache.org Received: (qmail 7003 invoked by uid 99); 25 Mar 2015 15:21:53 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 25 Mar 2015 15:21:53 +0000 Date: Wed, 25 Mar 2015 15:21:53 +0000 (UTC) From: "Devaraj K (JIRA)" To: yarn-issues@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (YARN-3225) New parameter or CLI for decommissioning node gracefully in RMAdmin CLI MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/YARN-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14380040#comment-14380040 ] Devaraj K commented on YARN-3225: --------------------------------- Thanks [~djp] for your comments and review. bq. There should be no difference for decommission forcelly and previous decommission as there should be no side effect to decommission a decommissioned node (API marked with Idempotent) If we make the refreshNodesForcefully as same as refreshNodes() API, there could be a chance of becoming node state incosistent if the hosts file updated with the newnodes or removal of existing nodes after the refreshNodesGracefully and before the refreshNodesForcefully. I feel we need to have difference that refreshNodes()/refreshNodesGracefully() should read the hosts file and refreshNodesForcefully() would not read the hosts file and only move the node state from DECOMMISSIONING to DECOMMISSIONED. bq. May be it is better to add getDecommissioningNodes() to return a list of decommissioning nodes instead of returning a boolean value here? We can print it out the decommissioning nodes that haven't finished (or a subset of them if large size) when hitting timeout at the end. It would be good idea to show the decommissioning nodes, I will include this in the new patch. > New parameter or CLI for decommissioning node gracefully in RMAdmin CLI > ----------------------------------------------------------------------- > > Key: YARN-3225 > URL: https://issues.apache.org/jira/browse/YARN-3225 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Junping Du > Assignee: Devaraj K > Attachments: YARN-3225-1.patch, YARN-3225.patch, YARN-914.patch > > > New CLI (or existing CLI with parameters) should put each node on decommission list to decommissioning status and track timeout to terminate the nodes that haven't get finished. -- This message was sent by Atlassian JIRA (v6.3.4#6332)