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é <j...@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+CarbonData+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 in
> > 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 <chenliang...@apache.org>
> wrote:
> >
> >> Hi all,
> >>
> >> PPMC vote for the release Apache CarbonData 0.2.0-incubating has passed.
> >> 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/RESULT-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-incubating/
> >>
> >> The git tag is:
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=incubator-carbondata.git;a=commit;h=848d3c03b63d2eba4160b25e60c097fc38aeb1e3
> >>
> >> Staging Repository:
> >>
> https://repository.apache.org/content/repositories/orgapachecarbondata-1006
> >>
> >> 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=12320220&version=12337896
> >>
> >> 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é
> jbono...@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Reply via email to