Mailing-List: contact general-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: general@incubator.apache.org
MIME-Version: 1.0
References: <CAM7YSv0vxNpwvEV_ejLtRPuQrpQ47Vdj6TUc6HpgJmjM5LzuCQ@mail.gmail.com>
 <CAOqetn9ok3XVC_BbE=-MAShw914Ndh3Y_9Djw5p6Y3SwkFJTtw@mail.gmail.com> <768327c3-d3f5-fcdd-492c-34d3d52e763b@nanthrax.net>
In-Reply-To: <768327c3-d3f5-fcdd-492c-34d3d52e763b@nanthrax.net>
From: "John D. Ament" <johndament@apache.org>
Date: Sat, 19 Nov 2016 12:55:25 +0000
Message-ID: <CAOqetn8ntt94m=vrT+Tjvyev7rzm4r6+RDdqW6SzAvDz6_DmPQ@mail.gmail.com>
Subject: Re: [VOTE] Apache CarbonData 0.2.0-incubating release
To: general@incubator.apache.org
Content-Type: multipart/alternative; boundary=94eb2c0afef64edc4d0541a6f03e
archived-at: Sat, 19 Nov 2016 12:55:39 -0000

--94eb2c0afef64edc4d0541a6f03e
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

JB -

I don't disagree with you.  The release contents are good, but doesn't
build from source.  When I asked how to build from source, I was given "mvn
clean -Pbuild-with-format install" without mention of the thrift binary
requirement.  Maybe a user will already have thrift, maybe they won't.  But
either way, they may not know how to build from source.  There was an
ensuing conversation about why carbondata-format was needed or not needed,
which didn't seem to help my situation.

Anyways now that I know thrift is required, I'm running a build with it
installed.

John

On Sat, Nov 19, 2016 at 4:03 AM Jean-Baptiste Onofr=C3=A9 <jb@nanthrax.net>
wrote:

> Hi John,
>
> AFAIR, the thrift use was required for CarbonData 0.1.0-incubating
> release. The 0.1.0-incubating passed with 3 +1 binding votes (Uma,
> Justin, and me).
>
> We agreed that the enforcement of the thrift dependency was not ideal,
> that's why we improved the build: the "old style" build is always
> possible (with thrift installed of your machine), and for convenience,
> we "pre-build" the file format: then the user can build on his machine
> without thrift (the file format is downloaded from Maven Central).
>
> In the README.md, we provide the build instructions:
>
>
> https://cwiki.apache.org/confluence/display/CARBONDATA/Building+CarbonDat=
a+And+IDE+Configuration
>
> we can see:
>
> $ mvn clean -DskipTests -Pbuild-with-format install
>
> and we can see thrift as prerequisites.
>
> IMHO, I think we think the build instructions are correct.
>
> As the build instruction are on the wiki (not in the release), if you
> consider that the build instructions should be improved, please let us
> know, we will update. However, as the build should work on your machine
> as soon as you have thrift installed, I don't think we should cancel
> this release.
>
> Thoughts ?
>
> Thanks !
> Regards
> JB
>
> On 11/14/2016 12:00 AM, John D. Ament wrote:
> > Hi,
> >
> > Can you please provide build instructions?  It fails for me on
> >
> > [ERROR] Failed to execute goal on project carbondata-core: Could not
> > resolve dependencies for project
> > org.apache.carbondata:carbondata-core:jar:0.2.0-incubating: Could not
> find
> > artifact org.apache.carbondata:carbondata-format:jar:0.2.0-incubating i=
n
> > pentaho-releases (http://repository.pentaho.org/artifactory/repo/) ->
> [Help
> > 1]
> >
> > I'm simply running "mvn clean install" as a WAG on how its intended to
> work.
> >
> > John
> >
> > On Sun, Nov 13, 2016 at 10:33 AM Liang Chen <chenliang613@apache.org>
> wrote:
> >
> >> Hi all,
> >>
> >> PPMC vote for the release Apache CarbonData 0.2.0-incubating has passe=
d.
> >> Now, we kindly submit this release to the IPMC.
> >>
> >> Here's the PPMC vote thread for 0.2.0 release:
> >>
> >>
> http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/RESUL=
T-VOTE-Apache-CarbonData-0-2-0-incubating-release-td2823.html
> >>
> >> The source distribution, with signatures is there:
> >>
> >>
> https://dist.apache.org/repos/dist/dev/incubator/carbondata/0.2.0-incubat=
ing/
> >>
> >> The git tag is:
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=3Dincubator-carbondata.git;a=3D=
commit;h=3D848d3c03b63d2eba4160b25e60c097fc38aeb1e3
> >>
> >> Staging Repository:
> >>
> https://repository.apache.org/content/repositories/orgapachecarbondata-10=
06
> >>
> >> The artifacts have been signed with this key:
> >> https://dist.apache.org/repos/dist/dev/incubator/carbondata/KEYS
> >>
> >> The Release Notes is:
> >>
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=3D123202=
20&version=3D12337896
> >>
> >> Please vote to approve this release:
> >>
> >> [ ] +1 Approve the release
> >> [ ] -1 Don't approve the release (please provide specific comments)
> >>
> >> This vote will be open for at least 72 hours.
> >>
> >> Thanks !
> >> Regards
> >>
> >
>
> --
> Jean-Baptiste Onofr=C3=A9
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

--94eb2c0afef64edc4d0541a6f03e--