Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread 金铸
+1 在 2016/8/20 2:57, Jean-Baptiste Onofré 写道: Hi all, I submit the first CarbonData release to your vote. Release Notes: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12320220=12337895 Staging Repository:

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread Uma Maheswara Rao Gangumalla
BTW, Justin has changed his vote to +1 now. Regards, Uma On Mon, Aug 22, 2016 at 11:54 PM, Uma Maheswara Rao Gangumalla < umaganguma...@gmail.com> wrote: > Oh good catch JB, > Seems RAT can't find if two headers...are we really having some > proprietary information still in the code? I think we

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread Uma Maheswara Rao Gangumalla
Oh good catch JB, Seems RAT can't find if two headers...are we really having some proprietary information still in the code? I think we need clean up and more reviews on that aspect. Yeah if we have from other open source projects then mentioning in NOTICE sounds good instead of copy. Regards,

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread Jean-Baptiste Onofré
If the files comes from OpenCV, then we should at least mention in the NOTICE. Good point Henry. Regards JB On 08/23/2016 08:15 AM, Henry Saputra wrote: RAT check seems to pass if you have 2 license headers. More important question is that, does CarbonData team copy some classes and

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread Henry Saputra
RAT check seems to pass if you have 2 license headers. More important question is that, does CarbonData team copy some classes and interfaces defintions from other projects like OpenCV [1] ? It is ASF license, but I am wondering why you need to copy instead of just OpenCV declaration instead? -

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-23 Thread Jean-Baptiste Onofré
FYI, the vote didn't pass at IPMC due to bad license header in the following files: 1. ./integration/spark/src/main/java/org/apache/carbondata/spark/partition/api/DataPartitioner.java 2. ./integration/spark/src/main/java/org/apache/carbondata/spark/partition/api/impl/CSVFilePartitioner.java

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-22 Thread Uma gangumalla
+1 (binding) DISCLAMER file exists Checksums looks good License header exists build is fine. Thanks all for the efforts in making the release. Regards, Uma On Mon, Aug 22, 2016 at 8:19 AM, Jean-Baptiste Onofré wrote: > Hi all, > > the PPMC vote to release Apache

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-22 Thread Henry Saputra
LICENSE file looks good DISCLAIMER file exists NOTICE file looks good Checksum files look good Sig file looks good No 3rd party exes in the source artifacts. License header exist +1 (Binding) - Henry On Mon, Aug 22, 2016 at 8:19 AM, Jean-Baptiste Onofré wrote: > Hi all, > >

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-22 Thread Liang Big data
Hi JB Thank you pointed out it, yes we will definitely avoid such kind of binary file in the source code. Regards Liang 2016-08-22 20:40 GMT+08:00 Jean-Baptiste Onofré : > +1 (binding) > > I checked: > - LICENSE, NOTICE, DISCLAIMER are there > - name contains incubating >

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-22 Thread Ravindra Pesala
Hi, Yes JB. We will avoid this binary file in the source, Actually we are using this binary for only running testcases . We will modify this testcase to generate binary file on fly and delete it once done. Thanks, Ravi. On 22 August 2016 at 18:10, Jean-Baptiste Onofré

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-20 Thread Venkata Gollamudi
August 19, 2016 7:33 PM > To: dev@carbondata.incubator.apache.org > Subject: Re: [VOTE] Apache CarbonData 0.1.0-incubating release > > +1 (binding) > > > > -- > View this message in context: > http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/VOTE-Apache-CarbonData-0-1-0-incu

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-20 Thread Vimal Das Kammath
+1 (binding) On Aug 20, 2016 12:27 AM, "Jean-Baptiste Onofré" wrote: > Hi all, > > I submit the first CarbonData release to your vote. > > Release Notes: > https://issues.apache.org/jira/secure/ReleaseNote.jspa?proje > ctId=12320220=12337895 > > Staging Repository: >

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-20 Thread Jacky Li
t; To: dev@carbondata.incubator.apache.org > Subject: Re: [VOTE] Apache CarbonData 0.1.0-incubating release > > +1 (binding) > > > -- > View this message in context: > http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/VOTE-Apache-CarbonData-0-1-0-incubating-rel

RE: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-19 Thread Jihong Ma
+1 (binding) Great work! Jihong -Original Message- From: chenliang613 [mailto:chenliang6...@gmail.com] Sent: Friday, August 19, 2016 7:33 PM To: dev@carbondata.incubator.apache.org Subject: Re: [VOTE] Apache CarbonData 0.1.0-incubating release +1 (binding) -- View this message

Re: [VOTE] Apache CarbonData 0.1.0-incubating release

2016-08-19 Thread Ravindra Pesala
+1 Thanks, Ravi, On Sat, 20 Aug 2016 8:05 am jarray, wrote: > +1 > > > Regards > jarray > On 08/20/2016 02:57, Jean-Baptiste Onofré wrote: > Hi all, > > I submit the first CarbonData release to your vote. > > Release Notes: > >