Sorry for wrong email address. I repeat the same message. Hi guys, +1 for Hyunsik and Jihun suggestions. We don't need to include test jars to the binary release.
Sincerely, Jihoon On Wed, Feb 25, 2015 at 2:07 PM Jihoon Son <[email protected]> wrote: > Hi guys, > +1 for Hyunsik and Jihun suggestions. We don't need to include test jars > to the binary release. > > Sincerely, > Jihoon > > > On Wed, Feb 25, 2015 at 10:46 AM Jinho Kim <[email protected]> wrote: > >> Guys, >> >> Test binary jars does not included in source tarball >> >> -Jinho >> Best regards >> >> 2015-02-25 10:25 GMT+09:00 Jaehwa Jung <[email protected]>: >> >> > I also agree with Jihun's opinion. :) >> > >> > >> > 2015-02-25 8:44 GMT+09:00 Jihun Kang <[email protected]>: >> > >> > > I agree with your suggestion. We don't need to include source files >> if we >> > > release source tarball and binary tarball. In addition, how about >> > excludes >> > > test jars from binary tarball? Test source files can be included in >> > source >> > > tarball, I think. >> > > 2015. 2. 25. 오전 5:01에 "Hyunsik Choi" <[email protected]>님이 작성: >> > > >> > > > Hi, >> > > > >> > > > The binary tarball is mostly used to launch Tajo clusters. So I >> think >> > > > that source jars in binary tarball itself is unnecessary in general. >> > > > In addition, we also provide source jars through maven, and most of >> > > > users will download and use source jars in this way. In my opinion, >> It >> > > > would be better if a binary tarball does not include source tarball. >> > > > >> > > > How do you think about that? >> > > > >> > > > - hyunsik >> > > > >> > > > On Mon, Feb 23, 2015 at 5:02 PM, Hyunsik Choi <[email protected]> >> > > wrote: >> > > > > This vote is canceled due to wrong version in some pom files. >> > > > > >> > > > > On Mon, Feb 23, 2015 at 5:00 PM, Hyunsik Choi <[email protected] >> > >> > > > wrote: >> > > > >> Hi guys, >> > > > >> >> > > > >> Thank you for your review. I'll cancel this release vote, and >> then >> > > > >> I'll call a vote for new release tarball after I reflect your >> > > > >> comments. >> > > > >> >> > > > >> Thanks, >> > > > >> Hyunsik >> > > > >> >> > > > >> On Sun, Feb 22, 2015 at 9:37 PM, Jinho Kim <[email protected]> >> > wrote: >> > > > >>> Hyunsik, >> > > > >>> >> > > > >>> In addition, I found unmodified pom.xml and sphinx doc. Could >> you >> > > > remove it >> > > > >>> ? >> > > > >>> Here is my grep : >> > > > >>> >> > > > >>> kimjh:tajo-0.10.0-src kimjh$ grep -r 'SNAPSHOT' ./ >> > > > >>> .//CHANGES: 2.6.0-SNAPSHOT. (jaehwa) >> > > > >>> .//pom.xml: <tajo.version>0.9.0-SNAPSHOT</tajo.version> >> > > > >>> .//tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml: >> > > > >>> <version>0.10.0-SNAPSHOT</version> >> > > > >>> .//tajo-common/src/main/java/org/apache/tajo/util/VersionInf >> o.java: >> > > * >> > > > >>> @return the Tajo version string, eg. "0.9.0-SNAPSHOT" >> > > > >>> .//tajo-docs/pom.xml: <version>0.9.0-SNAPSHOT</version> >> > > > >>> >> > .//tajo-docs/src/main/sphinx/configuration/catalog_configuration.rst: >> > > > $ ls >> > > > >>> tajo-dist/target/tajo-x.y.z-SNAPSHOT.tar.gz >> > > > >>> .//tajo-docs/src/main/sphinx/getting_started.rst: $ ls >> > > > >>> tajo-dist/target/tajo-x.y.z-SNAPSHOT.tar.gz >> > > > >>> .//tajo-docs/src/main/sphinx/getting_started.rst: $ tar xzvf >> > > > >>> ${TAJO_SRC}/tajo-dist/target/tajo-x.y.z-SNAPSHOT.tar.gz >> > > > >>> >> > > > >>> -Jinho >> > > > >>> Best regards >> > > > >>> >> > > > >>> 2015-02-23 14:27 GMT+09:00 Jaehwa Jung <[email protected]>: >> > > > >>> >> > > > >>>> Hi Guys >> > > > >>>> >> > > > >>>> I also agree with Jinho's opinion. It may be an issue for >> > installing >> > > > Tajo. >> > > > >>>> >> > > > >>>> Cheers >> > > > >>>> Jaehwa >> > > > >>>> >> > > > >>>> 2015-02-23 13:55 GMT+09:00 Jinho Kim <[email protected]>: >> > > > >>>> >> > > > >>>> > Thank you for your work. >> > > > >>>> > I found source jar in binary tarball. In my opinion, we >> should >> > > move >> > > > these >> > > > >>>> > jars to a source directory. because source jars included in >> > > > classpath. >> > > > >>>> > How do you think ? >> > > > >>>> > >> > > > >>>> > -Jinho >> > > > >>>> > Best regards >> > > > >>>> > >> > > > >>>> > 2015-02-21 9:40 GMT+09:00 Hyunsik Choi <[email protected]>: >> > > > >>>> > >> > > > >>>> > > Hi folks, >> > > > >>>> > > >> > > > >>>> > > I have created an Apache Tajo 0.10.0 release candidate >> (rc0). >> > > > >>>> > > >> > > > >>>> > > This is a major release which includes lots of new >> features, >> > bug >> > > > >>>> > > fixes, and improvements. Especially, this release includes >> > > > >>>> > > lightweight JDBC driver, HBase storage integration, hadoop >> > > > >>>> > > dependency removal from Tajo client, and >> > > > >>>> > > improved Amazon S3 support. >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > > The RC0 is available at: >> > > > >>>> > > https://dist.apache.org/repos/ >> dist/dev/tajo/tajo-0.10.0-rc0/ >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > > Nexus Staging URL: >> > > > >>>> > > >> > > > https://repository.apache.org/content/repositories/orgapache >> tajo-1002/ >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > > The RC0 tag in git is here: >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > >> > > > >>>> >> > > > >> > > >> > https://git-wip-us.apache.org/repos/asf?p=tajo.git;a=commit;h= >> 95f13b3a0ba651b1b67d126b06ce92a10cbe5eff >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > > Release notes is at: >> > > > >>>> > > >> > > > >>>> > >> > > > >>>> >> > > > >> > > >> > https://dist.apache.org/repos/dist/dev/tajo/tajo-0.10.0-rc0/ >> relnotes.html >> > > > >>>> > > >> > > > >>>> > > >> > > > >>>> > > and the KEYS file containing the PGP keys used to sign the >> > > > release can >> > > > >>>> > > currently be found at: >> > > > http://people.apache.org/keys/group/tajo.asc >> > > > >>>> > > >> > > > >>>> > > The vote will be open for at least 72 hours. >> > > > >>>> > > >> > > > >>>> > > Please vote: >> > > > >>>> > > >> > > > >>>> > > [ ] +1 release the RC0 as tajo-0.10.0 >> > > > >>>> > > [ ] + 0 no opinion >> > > > >>>> > > [ ] -1 do not release this package because ... >> > > > >>>> > > >> > > > >>>> > > Here is my +1. >> > > > >>>> > > >> > > > >>>> > > Best regards, >> > > > >>>> > > Hyunsik Choi >> > > > >>>> > > >> > > > >>>> > >> > > > >>>> >> > > > >> > > >> > >> >
