Hi John Thanks for your comments.
Because CarbonData is data format, for supporting multiple language(such as:Java,C++,C...) for satisfying different users, the source code(format files) could be compiled by Apache Thrift. As we discussed in the previous release session: 1. The build command need to be added for pointing out Thrift installation required: https://github.com/apache/incubator-carbondata/tree/master/build 2. For general users, they don't need to install Apache thrift, we already facilitated it as much as possible and uploaded “format dependency jar” to maven repo. Only during checking and preparing release, PPMC and IPMC need install Apache Thrift for compiling and checking source code, once the release be published, the format jar will be exist in maven repo. BTW: this is common issues for softwares which are integrating with Apache Thrift for supporting different code language. Regards Liang 2017-01-27 11:15 GMT+08:00 John D. Ament <johndam...@apache.org>: > LIang, > > Regarding the use of carbondata-format, during the last release this issue > came up as well. My understanding was that it was to be fixed in this > release. Is that no longer the case? > > John > > On Thu, Jan 26, 2017 at 10:04 PM Liang Chen <chenliang6...@gmail.com> > wrote: > > > Hi Justin > > > > Thanks for your checking. > > > > 1.Please install Apache Thrift 0.9.3 for compiling "carbondata-format", > > please find the build guide: > > https://github.com/apache/incubator-carbondata/tree/master/build > > (Remark:Once the release be published , general users don't need to > install > > Thrift, the carbondata-format dependency jar can be downloaded from maven > > repo.) > > > > 2.For missing license headers issue, already raised JIRA and will fix in > > the next release. > > https://issues.apache.org/jira/browse/CARBONDATA-682? > > jql=project%20%3D%20CARBONDATA > > > > 3.OK, will use apache email id to sign artifact for next release. > > > > Regards > > Liang > > > > > > 2017-01-27 10:03 GMT+08:00 Justin Mclean <jus...@classsoftware.com>: > > > > > Hi, > > > > > > +0 (binding) woudl change to +1 if I could compile > > > > > > I checked > > > - name includes incubating > > > - DISCLAIMER exists > > > - LICENSE and NOTICE good (year in NOTICE needs updating) > > > - No unexpected binaries > > > - A couple of source files are missing headers [1][2]. Can you fix this > > in > > > the next release release. > > > - Can't compile from source > > > > > > When compiling I get this error when trying to compile core. > > > [ERROR] Failed to execute goal on project carbondata-core: Could not > > > resolve dependencies for project > > org.apache.carbondata:carbondata-core:jar:1.0.0-incubating: > > > Failure to find > > org.apache.carbondata:carbondata-format:jar:1.0.0-incubating > > > in http://repo1.maven.org/maven2 was cached in the local repository, > > > resolution will not be reattempted until the update interval of central > > has > > > elapsed or updates are forced -> [Help 1] > > > > > > It would also be a good idea to sign the artefact with an apache email > > > address rather than a gmail one. > > > > > > Thanks, > > > Justin > > > > > > 1. ./integration/spark-common-test/src/test/scala/org/apache/ > > > carbondata/integration/spark/testsuite/primitiveTypes/Float > > > DataTypeTestCase.scala > > > 2. ./integration/spark-common-test/src/test/scala/org/apache/ > > > carbondata/spark/testsuite/datetype/DateTypeTest.scala > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > > > > > > > -- > > Regards > > Liang > > > -- Regards Liang
