Return-Path: X-Original-To: apmail-incubator-general-archive@www.apache.org Delivered-To: apmail-incubator-general-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 22FFF10D7C for ; Wed, 26 Mar 2014 00:37:01 +0000 (UTC) Received: (qmail 98649 invoked by uid 500); 26 Mar 2014 00:36:31 -0000 Delivered-To: apmail-incubator-general-archive@incubator.apache.org Received: (qmail 98223 invoked by uid 500); 26 Mar 2014 00:36:22 -0000 Mailing-List: contact general-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: general@incubator.apache.org Delivered-To: mailing list general@incubator.apache.org Received: (qmail 98195 invoked by uid 99); 26 Mar 2014 00:36:22 -0000 Received: from minotaur.apache.org (HELO minotaur.apache.org) (140.211.11.9) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 26 Mar 2014 00:36:22 +0000 Received: from localhost (HELO mail-wi0-f181.google.com) (127.0.0.1) (smtp-auth username apurtell, mechanism plain) by minotaur.apache.org (qpsmtpd/0.29) with ESMTP; Wed, 26 Mar 2014 00:36:21 +0000 Received: by mail-wi0-f181.google.com with SMTP id hm4so1256124wib.14 for ; Tue, 25 Mar 2014 17:36:20 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=uj6adADM/yY6ZiEpqOFaYIlYNQdqqf5i/bIQlncRWYQ=; b=mxao9qv5ncovZ7YSdnclR7HiMboRKmy50rBLWE+5WnpvfNH3FRW6fSVidnCr3Q9SvF yENDJVs32/xb5G3Ve8Amdf8r5aWa5QGYBNGdyJsC+kSfb2S5/TlMjs5Nbwmqh27W3i9P 93sV31nhgUtqPC84+skEZuNJj7FpMhURDedsWGGAYnvalzNVf9ynoZIkL2ewtDgudQvx k+ElBFIY9b+BE5/Xt/f1RsPpKbkny6Wq1d/mqjKLFxsC741tSWzfGS5/AajUZSEphpmD oTUeWrZjhY7ztxUpljueNluxxZq0lTvjcnIDa+PnvbgM2IEz4H9sKKbIzauqRbIwgUmZ 3Ftg== X-Received: by 10.180.210.171 with SMTP id mv11mr25307273wic.44.1395794180373; Tue, 25 Mar 2014 17:36:20 -0700 (PDT) MIME-Version: 1.0 Received: by 10.227.215.84 with HTTP; Tue, 25 Mar 2014 17:35:40 -0700 (PDT) In-Reply-To: References: From: Andrew Purtell Date: Tue, 25 Mar 2014 17:35:40 -0700 Message-ID: Subject: Re: [VOTE] Release of Apache Phoenix 3.0.0 incubating RC1 To: "general@incubator.apache.org" Content-Type: multipart/alternative; boundary=001a11c25d364d037c04f577a8a3 --001a11c25d364d037c04f577a8a3 Content-Type: text/plain; charset=ISO-8859-1 On Tue, Mar 25, 2014 at 4:25 PM, sebb wrote: > > Yes, we bundle ANTLR in our binary distribution. Most of the other items > > are pulled in based on the transitive dependencies of other jars we've > > bundled in our binary distribution. > > I see now why I did not notice the 3rd party binaries. > They seem to have been merged into jars which look like phoenix code - > and indeed also contain phoenix code. > > That is a very non-standard way to do things, and I think could > mislead end-users as to the provenance of the code. > > It's OK to bundle separate jars in a binary release (assuming > licensing etc is OK), but I don't think it's OK to merge 3rd party > code with ASF code in a single jar. > Apart from anything, that will play havoc with Maven and possibly > other dependency management systems. I believe this is done because Phoenix is a JDBC client, and JDBC drivers are usually packaged as single JARs for convenience. James could confirm or refute. I concluded this is acceptable practice having seen it elsewhere at Apache, for example in Apache Pig, their convenience fatjar artifact. -- Best regards, - Andy Problems worthy of attack prove their worth by hitting back. - Piet Hein (via Tom White) --001a11c25d364d037c04f577a8a3--