I think we need to fix this properly.
I recommend to just commit the generated Java Thrift code to make sure
people could build it without waiting for Maven central push.
- Henry
On Tue, Nov 15, 2016 at 3:50 PM, John D. Ament
wrote:
> Hi sorry but I'm -1 since I cannot build it from source. W
Hi sorry but I'm -1 since I cannot build it from source. Would be ok if it
compiles and some tests fail, but this won't even compile.
Please note - -1's are not veto's if you get enough +1's to outweigh, it
will still pass. Though it would be good to if you re-roll the release to
fix this build
Disclaimer exist
LICENSE.txt exist
Signature file fine
Contains incubating
I think Readme.md file can have section about Prerequisites and point to
https://cwiki.apache.org/confluence/display/CARBONDATA/Building+CarbonData+
And+IDE+Configuration
>From Wiki
==
I
Hi Liang, Ravindran,
>From release artifacts it looks good:
Hash files look good
Signature file looks good
Disclaimer file exist
No 3rd party exes in release source artifact
+1
Should probably add in the README to install Thrift to install locally.
- Henry
On Tue, Nov 15, 2016 at 6:46 AM, Lian
Hi John D. Ament and Henry
Gentle reminder!
Could you please check again? please feel free to let me know if need to
provide more information.
BTW,building document at:
https://cwiki.apache.org/confluence/display/CARBONDATA/Building+CarbonData+And+IDE+Configuration
Regards
Liang
--
View th
Thanks to all who voted. The following IPMC binding votes were cast:
+1 Gary Gregory
+1 Sergio Fernández
+1 John D Ament
The release candidate is therefore ACCEPTED.
I will proceed with publishing the release.
On 11 Nov 2016 11:03 pm, "Stian Soiland-Reyes" wrote:
> The Apache Commons RDF podl