incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jean-Baptiste Onofré ...@nanthrax.net>
Subject Re: [VOTE] Apache CarbonData 0.2.0-incubating release
Date Mon, 21 Nov 2016 06:29:07 GMT
Agree John: we will add more explicitly the thrift binary installation 
requirement in the build instructions.

Thanks !

Regards
JB

On 11/19/2016 01:55 PM, John D. Ament wrote:
> 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é <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+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 <chenliang613@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é
>> 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
>>
>>
>

-- 
Jean-Baptiste Onofré
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


Mime
View raw message