+1 for releasing only source code On Wed, Dec 5, 2018 at 6:01 PM Taegeon Um <taegeo...@gmail.com> wrote:
> Thanks Joo Yeon! > > > On 3 Dec 2018, at 6:16 PM, Joo Yeon Kim <jooy...@apache.org> wrote: > > > > Thank you mentors for the feedback and the vote! > > > > I've received a few requests to include the features merged to master > after > > rc1, if possible. > > It looks like I'll be setting up another tarball as per Gon's request > > anyway, so let's go ahead with another release candidate. > > Apologies to Davor, Jean-Baptiste and Gon for the inconvenience, it looks > > like we need to vote again :( > > > > Regarding the comments we received: > > 1. I've filed a PR to check license headers, and it will be included in > rc2. > > 2. @developers, > > What are your opinions about Davor's comment? > > > > > > +1 for release source-only. > > > > > > > > > *-> Examples produce shaded artifact, which bundles parts of Beam, > > Spark.This bundling requires updates to license and notice files. But, > > thisshading process doesn't change what the main output JAR is, or what > > isbeing uploaded to Nexus. I'm not sure what's the value of releasing > > binaryexamples when nobody should link against them. I'd guess you'd want > > tochange this either to a Maven archetype or release source-only. * > > > > Best, > > Joo Yeon > > > > On Mon, Dec 3, 2018 at 7:06 AM Byung-Gon Chun <bgc...@gmail.com> wrote: > > > >> +1 (binding) > >> > >> Great work! PGP and checksums look good. > >> clean-install on Mac OS X/Java 1.8 works fine. > >> > >> Joo Yeon, one minor improvement to add. > >> The files in the tarball contain ^M (extra CR) characters. > >> I'm guessing you probably generated the tar ball on Windows. > >> > >> Best, > >> Gon > >> > >> > >> On Mon, Dec 3, 2018 at 4:19 AM Jean-Baptiste Onofré <j...@nanthrax.net> > >> wrote: > >> > >>> +1 (binding) > >>> > >>> Good work, I just did a quick review. As mentioned by Davor, there are > >>> some minor improvements to do for the next release. > >>> > >>> Regards > >>> JB > >>> > >>> On 02/12/2018 19:40, Davor Bonaci wrote: > >>>> +1 (binding) > >>>> > >>>> Great job! > >>>> > >>>> Things to improve, none of which I consider release-blocking: > >>>> > >>>> - Missing Apache header in a few files, mostly XML and like files. It > >>> would > >>>> be great to make all that part of checkstyle. > >>>> > >>>> - Examples produce shaded artifact, which bundles parts of Beam, > Spark. > >>>> This bundling requires updates to license and notice files. But, this > >>>> shading process doesn't change what the main output JAR is, or what is > >>>> being uploaded to Nexus. I'm not sure what's the value of releasing > >>> binary > >>>> examples when nobody should link against them. I'd guess you'd want to > >>>> change this either to a Maven archetype or release source-only. > >>>> > >>>> You can choose to address (or push back against) these for this > release > >>> or > >>>> the next subsequent release. The first requires a new vote, so I > >> wouldn't > >>>> recommend fixing it now, unless there are other issues. Depending on > >> how > >>>> you want to address the second one, you may be able to fix it within > >>> Nexus, > >>>> without a new candidate/vote. > >>>> > >>>>> If this is incorrect/less-preferable, please let me know so that I > can > >>>>>> upload a new tarball. > >>>>> > >>>> > >>>> No, leave this as is. It is perfect. > >>>> > >>>> * * * > >>>> > >>>> Overall, I'm thrilled to see this release going out. Thank you for the > >>>> great work. > >>>> > >>> > >> > >> > >> -- > >> Byung-Gon Chun > >> > > -- Byung-Gon Chun