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 6EC6D200CA8 for ; Thu, 15 Jun 2017 10:09:59 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 6D51C160BDF; Thu, 15 Jun 2017 08:09:59 +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 B6079160BC4 for ; Thu, 15 Jun 2017 10:09:58 +0200 (CEST) Received: (qmail 15809 invoked by uid 500); 15 Jun 2017 08:09:57 -0000 Mailing-List: contact dev-help@asterixdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@asterixdb.apache.org Delivered-To: mailing list dev@asterixdb.apache.org Received: (qmail 15797 invoked by uid 99); 15 Jun 2017 08:09:56 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 15 Jun 2017 08:09:56 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 4505E1AFDDD for ; Thu, 15 Jun 2017 08:09:56 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -0.401 X-Spam-Level: X-Spam-Status: No, score=-0.401 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=2, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-2.8, RCVD_IN_SORBS_SPAM=0.5, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd2-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id ZVb--9TrZlEo for ; Thu, 15 Jun 2017 08:09:55 +0000 (UTC) Received: from mail-io0-f179.google.com (mail-io0-f179.google.com [209.85.223.179]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTPS id AF4DC5F6BF for ; Thu, 15 Jun 2017 08:09:54 +0000 (UTC) Received: by mail-io0-f179.google.com with SMTP id k93so7222980ioi.2 for ; Thu, 15 Jun 2017 01:09:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=0QUIeYCaxXu52FQMo0yqCPlPZiUt3lazwZAtS0xcARk=; b=iZcQPBb29r0lUAj/j7+NTJcr9CxaZ3BiPABlCU36V3FJSvB0zFHuVi3v5OfjA1mn0I Bzmr2/cdb3NeqTMB9RamPBS7bYU8cv/9FoEJILSO3Cb5vjpDQFQISyL46lZ1TdHIhhsW gyLootQRMu5l5WdHz5S+D0+FyJgNot0D6ctVkY0TBVuLm/CMQgPjrB7TOFborXpYTKCs SoLariLE+WK4Xu+35W19dBXQJk2tt052WWyJSEVrAIbzrD7dJzCiTg+OJUuiy9UDSfCR rZ+ERA7wLBPrlR0GcKS3g05+TwjYgfTQkj07Epl2aSIMnCYmI4uOETX9dmZXsFLQ5VnA 5Y3g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=0QUIeYCaxXu52FQMo0yqCPlPZiUt3lazwZAtS0xcARk=; b=YZpQn+3GAfYpLc9ADt+VjxSYUHfurd/lV6AiQqU/P01Iw8UDZyvyHNho3mS+Ec1Acp TULLKKei4sbwcp7XtvK+A4V7IcEI4QNUYnla8DN7cJ8JzHh/cE+FyX27xylLo8v1WcFE GpV0PkAuEhzJWIfK4q+Xm0gXuVWs8zXnTDtUl37f0u85w/1L0oBBFFGtZjinkZ2S3DPM SWIws8uFQHxnTjZ4VgPIqODc4gJiUkg1J8clKMKL+d3fwb+8h/zGspzWXNnti8Sw59FR /uIZGpB//xQhdE+rXJiRTuBATDqiaSI09VL/KKDUqHAm+Y+MdhzNC+hUopaC4GphqAFQ i0Sw== X-Gm-Message-State: AKS2vOz67fOanWtuoCPTSnvyI5GR9kpfgDHleMhTGrFYqt3okuWknZ2V p/G1D3uold4Hh30oztu1ythbYBC+Rq4Z X-Received: by 10.107.171.67 with SMTP id u64mr3665196ioe.125.1497514193914; Thu, 15 Jun 2017 01:09:53 -0700 (PDT) MIME-Version: 1.0 Received: by 10.79.85.139 with HTTP; Thu, 15 Jun 2017 01:09:53 -0700 (PDT) In-Reply-To: <96b581ce-8650-9ef3-a7d1-db78b9acf73d@gmail.com> References: <6DB13094-338D-4C68-B4D8-D14E06E1BEED@apache.org> <96b581ce-8650-9ef3-a7d1-db78b9acf73d@gmail.com> From: Yingyi Bu Date: Thu, 15 Jun 2017 01:09:53 -0700 Message-ID: Subject: Re: Commit messages To: dev@asterixdb.apache.org Content-Type: multipart/alternative; boundary="94eb2c059e149364700551fb31aa" archived-at: Thu, 15 Jun 2017 08:09:59 -0000 --94eb2c059e149364700551fb31aa Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable +1! Best, Yingyi On Wed, Jun 14, 2017 at 10:43 PM, Mike Carey wrote: > +1 !!! > > I think this is a GREAT proposal, and we can also then hopefully do the > equivalent of grep'ing the commits to identify things that we might want = to > incorporate in a high-level set of release notes. I also really like the > "no" requirement. Also, storage changes should really NOT be taken lightl= y > - they seriously inconvenience our customers and will hopefully cause the > tests to fail (the ones that check for backward-compat) - I would like to > set storage changes actually be voted on, ideally, if we could make that > part of our operating procedure somehow? > > Cheers, > > Mike > > > > On 6/14/17 9:15 AM, Till Westmann wrote: > >> Hi, >> >> some of us had a discussion with an SDSC team last week that is running = an >> AsterixDB instance. Their customer perspective on AsterixDB highlighted = a >> few areas of improvement to ease the consumption of AsterixDB. >> >> One thing that I=E2=80=99d like to follow up on are release notes. So fa= r we >> didn=E2=80=99t >> provide them and so changes to the system came as a surprise to everybod= y >> who is not monitoring commits closely. >> >> As I think that it=E2=80=99s not easy to provide good release notes I=E2= =80=99d like to >> propose some additions to our commit messages to ease the creation of >> release messages: >> >> Each commit message should >> 1) reference 1 or more JIRA issues (that hopefully provide a rationale f= or >> the change). >> 2) contain a description of changes to the user model (language syntax, >> configuration parameters, ..) >> 3) contain a description of storage format changes (that would require >> reloading or upgrading) >> 4) contain a description of interface changes (for source code consumers= ) >> >> and all reviewers should check that these are mentioned in the commit >> message. To increase the probability to we won=E2=80=99t forget to menti= on the >> changes in 2-4, I think that we should should explicitly mention the >> absence >> of such changes, i.e.: >> >> user model changes: no >> storage format changes: no >> interface changes: no >> >> Thoughts/concerns about this? >> Is this manageable? >> Are there other kinds of changes that have a high impact on consumers th= at >> we should call out? >> >> Cheers, >> Till=C2=B4 >> > > --94eb2c059e149364700551fb31aa--