From dev-return-15055-archive-asf-public=cust-asf.ponee.io@arrow.apache.org Tue Oct 1 01:06:36 2019 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 [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id 7666C180656 for ; Tue, 1 Oct 2019 03:06:36 +0200 (CEST) Received: (qmail 97230 invoked by uid 500); 1 Oct 2019 01:06:34 -0000 Mailing-List: contact dev-help@arrow.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@arrow.apache.org Delivered-To: mailing list dev@arrow.apache.org Received: (qmail 97219 invoked by uid 99); 1 Oct 2019 01:06:34 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd4-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Oct 2019 01:06:34 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd4-us-west.apache.org (ASF Mail Server at spamd4-us-west.apache.org) with ESMTP id C2388C20A4 for ; Tue, 1 Oct 2019 01:06:33 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd4-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 2.235 X-Spam-Level: ** X-Spam-Status: No, score=2.235 tagged_above=-999 required=6.31 tests=[KAM_NUMSUBJECT=0.5, KHOP_HELO_FCRDNS=0.399, RDNS_DYNAMIC=0.363, SPF_HELO_NONE=0.001, SPF_SOFTFAIL=0.972] autolearn=disabled Received: from mx1-he-de.apache.org ([10.40.0.8]) by localhost (spamd4-us-west.apache.org [10.40.0.11]) (amavisd-new, port 10024) with ESMTP id TRovCXK_f9a2 for ; Tue, 1 Oct 2019 01:06:31 +0000 (UTC) Received-SPF: Softfail (mailfrom) identity=mailfrom; client-ip=2401:2500:102:3039:153:126:206:245; helo=mail.clear-code.com; envelope-from=kou@clear-code.com; receiver= Received: from mail.clear-code.com (153.126.206.245.v6.sakura.ne.jp [IPv6:2401:2500:102:3039:153:126:206:245]) by mx1-he-de.apache.org (ASF Mail Server at mx1-he-de.apache.org) with ESMTPS id 714CC7DDF7 for ; Tue, 1 Oct 2019 01:06:29 +0000 (UTC) Received: from localhost (119.176.210.220.dy.bbexcite.jp [220.210.176.119]) by mail.clear-code.com (Postfix) with ESMTPSA id AC59C14670B for ; Tue, 1 Oct 2019 10:05:34 +0900 (JST) Date: Tue, 01 Oct 2019 10:05:20 +0900 (JST) Message-Id: <20191001.100520.170859175595748172.kou@clear-code.com> To: dev@arrow.apache.org Subject: Re: Subject: [VOTE] Release Apache Arrow 0.15.0 - RC1 From: Sutou Kouhei In-Reply-To: References: <20190930.063628.622936056438614978.kou@clear-code.com> X-Mailer: Mew version 6.8 on Emacs 26.1 Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: AC59C14670B X-Spamd-Result: default: False [1.40 / 999.00]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MV_CASE(0.50)[]; MIME_GOOD(-0.10)[text/plain]; TO_DN_NONE(0.00)[]; RCPT_COUNT_ONE(0.00)[1]; MID_CONTAINS_FROM(1.00)[]; RCVD_COUNT_ZERO(0.00)[0]; FROM_EQ_ENVFROM(0.00)[]; MIME_TRACE(0.00)[0:+]; ASN(0.00)[asn:2497, ipnet:220.210.128.0/18, country:JP] X-Rspamd-Server: mail.clear-code.com If we don't care the rustfmt check for release, how about removing the check from dev/release/verify-release-candidate.sh? In "Re: Subject: [VOTE] Release Apache Arrow 0.15.0 - RC1" on Sun, 29 Sep 2019 17:40:20 -0700, Andy Grove wrote: > Actually, I think the RC was cut just before 1.40.0 nightly was released, > which would explain why the rustfmt check fails now. As I think you already > said, it doesn't really matter anyway since it is just a formatting > difference. > > On Sun, Sep 29, 2019 at 2:44 PM Sutou Kouhei wrote: > >> I used dev/release/verify-release-candidate.sh. It install >> Rust automatically. >> Should we update >> >> https://github.com/apache/arrow/blob/master/dev/release/verify-release-candidate.sh#L452 >> ? >> >> In >> "Re: Subject: [VOTE] Release Apache Arrow 0.15.0 - RC1" on Sun, 29 Sep >> 2019 07:21:27 -0600, >> Andy Grove wrote: >> >> > Just fyi on the rustfmt issue, the formatting was recently updated for >> rust >> > 1.40 nightly and if you are using an older version the formatting check >> > will fail. >> > >> > On Sun, Sep 29, 2019, 5:56 AM Wes McKinney wrote: >> > >> >> It's up to Micah as RM, but I think it would be good to fix the >> sig-related >> >> issues or we may be dealing with "bug" reports until the next release. >> I'll >> >> work on source verification later today in the meantime to see if any >> other >> >> issues turn up >> >> >> >> On Sun, Sep 29, 2019, 1:19 AM Sutou Kouhei wrote: >> >> >> >> > -0 (binding) >> >> > >> >> > I ran the followings on Debian GNU/Linux sid: >> >> > >> >> > * TEST_CSHARP=0 \ >> >> > TEST_GLIB=0 \ >> >> > TEST_RUBY=0 \ >> >> > TEST_RUST=0 \ >> >> > JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64 \ >> >> > CUDA_TOOLKIT_ROOT=/usr \ >> >> > dev/release/verify-release-candidate.sh source 0.15.0 1 >> >> > * dev/release/verify-release-candidate.sh binaries 0.14.1 0 >> >> > >> >> > with: >> >> > >> >> > * gcc (Debian 9.2.1-7) 9.2.1 >> >> > * openjdk version "1.8.0_212" >> >> > * Node.JS v12.1.0 >> >> > * go version go1.12.9 linux/amd64 >> >> > * nvidia-cuda-dev 10.1.105-3 >> >> > >> >> > >> >> > I got the following failures: >> >> > >> >> > * Not ignorable: >> >> > * Binary: Bad signature >> >> > * centos-rc/6/Source/repodata/repomd.xml is failed >> >> > * We can't ignore this if removing the file from >> >> > https://bintray.com/apache/arrow/centos-rc/0.15.0-rc1 and >> >> > re-uploading it doesn't solve this problem. >> >> > >> >> > * Ignorable: >> >> > * C GLib and Ruby: Buildable but can't run test with GLib 2.62.0. >> >> > * It's caused by gobject-introspection gem. >> >> > * This is a known problem and not a C GLib problem. >> >> > * We can ignore this. (I'm fixing gobject-introspection gem.) >> >> > * Rust: "cargo +stable fmt --all -- --check" is failed (*) >> >> > * If I commented the command line out, Rust verification is >> passed. >> >> > * We can ignore this. Because this is just a lint error. >> >> > * C#: "sourcelink test" is failed >> >> > * We can ignore this. This is happened when we release 0.14.1 >> too. >> >> > * APT and Yum: arm64 and aarch64 are broken >> >> > * We can ignore this. >> >> > >> >> > (*) >> >> > ---- >> >> > + cargo +stable fmt --all -- --check >> >> > Diff in >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/arrow/src/array/ >> >> > builder.rs at line 1458: >> >> > let mut builder = StructBuilder::new(fields, field_builders); >> >> > assert!(builder.field_builder::(0).is_none()); >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/arrow/src/ >> >> > bitmap.rs at line 126: >> >> > assert_eq!(true, bitmap.is_set(6)); >> >> > assert_eq!(false, bitmap.is_set(7)); >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/ >> >> > aggregate.rs at line 1471: >> >> > ds, >> >> > ))))) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/ >> >> > context.rs at line 682: >> >> > >> >> > Ok(ctx) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/physical_plan/ >> >> > hash_aggregate.rs at line 720: >> >> > >> >> > Ok(()) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/physical_plan/ >> >> > merge.rs at line 134: >> >> > >> >> > Ok(()) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/physical_plan/ >> >> > projection.rs at line 171: >> >> > >> >> > Ok(()) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/physical_plan/ >> >> > selection.rs at line 180: >> >> > >> >> > Ok(()) >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/ >> >> > projection.rs at line 141: >> >> > >> >> > assert_eq!("c1", batch.schema().field(0).name()); >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > >> >> >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/execution/ >> >> > table_impl.rs at line 315: >> >> > true, >> >> > ); >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/ >> >> > logicalplan.rs at line 641: >> >> > println!("plan: {:?}", plan1); >> >> > }); >> >> > } >> >> > - >> >> > } >> >> > >> >> > Diff in >> >> > /tmp/arrow-0.15.0.tGMnP/apache-arrow-0.15.0/rust/datafusion/src/sql/ >> >> > planner.rs at line 569: >> >> > } >> >> > } >> >> > } >> >> > - >> >> > } >> >> > ---- >> >> > >> >> > >> >> > >> >> > Thanks, >> >> > -- >> >> > kou >> >> > >> >> > In < >> CAK7Z5T9mamQWtObWmZMJV_tMhAnWGMtndpCmE+_RPQN3LsEKmQ@mail.gmail.com> >> >> > "Subject: [VOTE] Release Apache Arrow 0.15.0 - RC1" on Fri, 27 Sep >> 2019 >> >> > 02:18:05 -0600, >> >> > Micah Kornfield wrote: >> >> > >> >> > > Hi, >> >> > > >> >> > > >> >> > > I would like to propose the following release candidate (RC1) of >> Apache >> >> > > >> >> > > Arrow version 0.15.0. This is a release consisting of 672 >> >> > > >> >> > > resolved JIRA issues[1]. (RC0 was aborted due to release script >> >> issues). >> >> > > >> >> > > >> >> > > This release candidate is based on commit: >> >> > > >> >> > > 77f091888dbe5a4e023a66b986a2dd474696061a [2] >> >> > > >> >> > > >> >> > > The source release rc1 is hosted at [3]. >> >> > > >> >> > > The binary artifacts are hosted at [4][5][6][7]. >> >> > > >> >> > > The changelog is located at [8]. >> >> > > >> >> > > >> >> > > Please download, verify checksums and signatures, run the unit >> tests, >> >> > > >> >> > > and vote on the release. See [9] for how to validate a release >> >> candidate. >> >> > > >> >> > > >> >> > > The vote will be open for at least 72 hours. >> >> > > >> >> > > >> >> > > [ ] +1 Release this as Apache Arrow 0.15.0 >> >> > > >> >> > > [ ] +0 >> >> > > >> >> > > [ ] -1 Do not release this as Apache Arrow 0.15.0 because... >> >> > > >> >> > > >> >> > > [1]: >> >> > > >> >> > >> >> >> https://issues.apache.org/jira/issues/?jql=project%20%3D%20ARROW%20AND%20status%20in%20%28Resolved%2C%20Closed%29%20AND%20fixVersion%20%3D%200.15.0 >> >> > > >> >> > > [2]: >> >> > > >> >> > >> >> >> https://github.com/apache/arrow/tree/77f091888dbe5a4e023a66b986a2dd474696061a >> >> > > >> >> > > [3]: >> >> > https://dist.apache.org/repos/dist/dev/arrow/apache-arrow-0.15.0-rc1 >> >> > > >> >> > > [4]: https://bintray.com/apache/arrow/centos-rc/0.15.0-rc1 >> >> > > >> >> > > [5]: https://bintray.com/apache/arrow/debian-rc/0.15.0-rc1 >> >> > > >> >> > > [6]: https://bintray.com/apache/arrow/python-rc/0.15.0-rc1 >> >> > > >> >> > > [7]: https://bintray.com/apache/arrow/ubuntu-rc/0.15.0-rc1 >> >> > > >> >> > > [8]: >> >> > > >> >> > >> >> >> https://github.com/apache/arrow/blob/77f091888dbe5a4e023a66b986a2dd474696061a/CHANGELOG.md >> >> > > >> >> > > [9]: >> >> > > >> >> > >> >> >> https://cwiki.apache.org/confluence/display/ARROW/How+to+Verify+Release+Candidates >> >> > >> >> >>