Thank you for your suggestion, Khurrum. Comment inline. I'd like to collect more feedbacks and then cut another RC.
On Mon, Jan 9, 2017 at 11:43 AM, Khurrum Nasim <khurrumnas...@gmail.com> wrote: > Thanks Sijie for working on this. A couple of comments: > > - do we need to publish artifacts for both scala 2.10 and 2.11 (probably > also 2.12)? as DL uses some scala libraries as the dependencies. > Ah, ah, good catch. We need to publish both 2.10 and 2.11. > - the source jar is also missing. do we need it? it is usually useful to > have a source jar so that people who uses any IDE tool can easily navigate > to the source code. > It is a good suggestion. We don't have source maven plugin configured. We need to configure that. > > - KN > > On Sun, Jan 8, 2017 at 8:04 PM, Sijie Guo <si...@apache.org> wrote: > > > Hi all, > > > > Please review and vote on the release candidate #1 for the version 0.4.0, > > as follows: > > > > [ ] +1, Approve the release > > [ ] -1, Do not approve the release (please provide specific comments) > > > > The complete staging area is available for your review, which includes: > > > > * JIRA release notes [1], > > * the official Apache source release to be deployed to > dist.apache.org > > [2], > > * all artifacts to be deployed to the Maven Central Repository [3], > > * source code tag "v0.4.0-RC1" [4], > > * website pull request listing the release [5] and publishing the API > > reference manual. > > > > A simple instruction for validation the source and binary packages. > > > > - source package: run "*./scripts/integration/smoketest.sh*" after > > building > > the package with "*mvn clean apache-rat:check package findbugs:check > > -DskipTests*" > > > > The vote will be open for at least 72 hours. It is adopted by majority > > approval, with at least 3 PPMC affirmative votes. > > > > Thanks, > > Sijie > > > > [1] > > https://issues.apache.org/jira/secure/ReleaseNote.jspa? > > projectId=12320620&version=12337980 > > [2] > > https://dist.apache.org/repos/dist/dev/incubator/distributedlog/0.4.0- > > incubating/ > > [3] > > https://repository.apache.org/content/repositories/ > > orgapachedistributedlog-1001/ > > [4] > > https://github.com/apache/incubator-distributedlog/tree/ > > v0.4.0-incubating-RC1 > > [5] https://github.com/apache/incubator-distributedlog/pull/109 > > >