Return-Path: X-Original-To: apmail-hadoop-common-dev-archive@www.apache.org Delivered-To: apmail-hadoop-common-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 24136F6F2 for ; Thu, 11 Apr 2013 12:36:50 +0000 (UTC) Received: (qmail 26797 invoked by uid 500); 11 Apr 2013 12:36:47 -0000 Delivered-To: apmail-hadoop-common-dev-archive@hadoop.apache.org Received: (qmail 26105 invoked by uid 500); 11 Apr 2013 12:36:40 -0000 Mailing-List: contact common-dev-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: common-dev@hadoop.apache.org Delivered-To: mailing list common-dev@hadoop.apache.org Received: (qmail 26055 invoked by uid 99); 11 Apr 2013 12:36:39 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Apr 2013 12:36:39 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of tucu@cloudera.com designates 209.85.223.171 as permitted sender) Received: from [209.85.223.171] (HELO mail-ie0-f171.google.com) (209.85.223.171) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Apr 2013 12:36:34 +0000 Received: by mail-ie0-f171.google.com with SMTP id e14so1920972iej.2 for ; Thu, 11 Apr 2013 05:36:14 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:mime-version:in-reply-to:references:from:date:message-id :subject:to:cc:content-type:x-gm-message-state; bh=JcEi+DTu1Ip2xAs4WZy4vCL3QZ6VuznOJgHljv4t2Yw=; b=djeZXcUtYgWOLPgtEWyvlaxe5YAPSgoa0QaB4OB2G6TgExnwOI/lDZfrdLztiLNRQx QJwim5MuCWLlixQ4wDCPXI/bjWvHcUI+uCepB4G6kBEhK2im/jbczK4Oml357d1xZxQA hkus5AUXYga1hKHuiAMH5EITzxzYBPrzyDv5A+Sytg+yEBoxVfMspWQ+meKfUA4UhOB+ nwPpnAWkyijOtStlD2yiKjYm7fc/wSTZM3Ltt2uG1yGRRJvRXgRlRzIBFofXOgJWKb8L qC5jBwspkPB9lIcFvx964zaoY1RM/TGmgezpDuLxc5R8vWXpnJU/h4oZsDiZQDfF5iYl 0tzQ== X-Received: by 10.50.128.47 with SMTP id nl15mr4556327igb.5.1365683774263; Thu, 11 Apr 2013 05:36:14 -0700 (PDT) MIME-Version: 1.0 Received: by 10.231.173.193 with HTTP; Thu, 11 Apr 2013 05:35:43 -0700 (PDT) In-Reply-To: References: <8BFFEEA2-E037-4C77-82F9-7A6CD63ECA3D@apache.org> From: Alejandro Abdelnur Date: Thu, 11 Apr 2013 05:35:43 -0700 Message-ID: Subject: Re: CHANGES.txt out of sync in the different branches To: "mapreduce-dev@hadoop.apache.org" Cc: "common-dev@hadoop.apache.org" Content-Type: multipart/alternative; boundary=089e013a286a3da5a004da1508cc X-Gm-Message-State: ALoCoQmqONNbBEnd6ostSUDb8W+zKmZTuqjzjnHsjcZwMiKecZCZBM/rY/tOK83wZ+5MGITMR/Jh X-Virus-Checked: Checked by ClamAV on apache.org --089e013a286a3da5a004da1508cc Content-Type: text/plain; charset=ISO-8859-1 Thanks for taking care of this Sid. Agree that using jira fix versions would be easier as a way to generate the changes. It would require some proper handling by the committer. For example, if something is committed to trunk(3.0.0) and branch-2(2.0.5) it would have fixedVersion 2.0.5, if later it is backported to a soon to be release 2.0.4, the fixVersion must be updated to 2.0.4. Another scenario is when you have different release lines, i.e 1.x and 2.x (or 2.0.x and 2.1.x), then the fixVersion should contain each line version. Still, IMO, much better than dealing with CHANGES.txt in different branches. Thanks again On Thu, Apr 11, 2013 at 12:44 AM, Siddharth Seth wrote: > I went ahead and committed a couple of changes to trunk and branch-2 to fix > the MR CHANGES.txt mess. Alexandro, I believe there's a couple of jiras > where you were waiting for CHANGES.txt fixes before merging to branch-2. > Not sure about past discussions, but can we re-visit removing CHANGES.txt > in favour of jira fix versions. > > While we maintain this file, a couple of points to prevent this ... > - For changes that go into 0.23, the CHANGES.txt entry needs to be included > under the 2.x line as well since 2.x and 0.23 are on a different release > schedule. > - When merging into a branch, please update CHANGES.txt on the src branch > as well. trunk was out of sync on a lot of CHANGES which have gone into > 2.0.5. > - Something going wrong with these files isn't always discovered > immediately, but adds up fast - so a little extra caution with changes, > merge-conflicts etc on these files... > > There's jiras like MAPREDUCE-4790, for which the fix version on jira is > trunk-win, but the patch is already in trunk. Does fix version trunk-win > imply the patch is in trunk as well ? > > Thanks > - Sid > > > On Wed, Apr 3, 2013 at 3:02 PM, Vinod Kumar Vavilapalli < > vinodkv@hortonworks.com> wrote: > > > I started looking at this yesterday, MAPREDUCE CHANGES.txt is completely > > broken. I'll try to fix it and then send out a note once I am done. > > > > Thanks, > > +Vinod > > > > On Apr 1, 2013, at 11:46 PM, Vinod Kumar Vavilapalli wrote: > > > > > > > > I've been looking at YARN and it seems to be fine. I presume common and > > hdfs too. > > > > > > MR clearly has issues. Have to manually fix it. Will do something > > tomorrow first thing. > > > > > > Thanks, > > > +Vinod Kumar Vavilapalli > > > > > > On Apr 1, 2013, at 3:53 PM, Alejandro Abdelnur wrote: > > > > > >> while trying to commit MAPREDUCE-5113 to branch-2 I've noticed that > the > > >> CHANGES.txt are out of sync. Commit message are on the wrong releases. > > >> > > >> I've spent some some time trying to fix it, but I did not find it > > straight > > >> forward to do so. > > >> > > >> I assume the same may be true for common, hdfs and yarn. > > >> > > >> I know why we use CHANGES.txt files has been discussed in the past, so > > I'll > > >> not raise a suggestion to get rid of them. > > >> > > >> Does anybody has a simple list of steps to fix this ? > > >> > > >> Thx > > >> > > >> -- > > >> Alejandro > > > > > > > > -- Alejandro --089e013a286a3da5a004da1508cc--