On Mon, Mar 30, 2020 at 12:00 AM Tzu-Li (Gordon) Tai <tzuli...@apache.org> wrote:
> @Hequn Cheng <chenghe...@gmail.com> > Good catches again! > > Regarding the incorrect versions: > I think technically those would not be hard blockers, since what matters > is their inclusion and licenses being acknowledged. > It would still be good to fix those though - could you open a ticket for > those? > Here, I mean to fix those in future bugfix releases (if this RC does indeed pass the vote as the official release). > > Regarding the names of the Python dists - > the name of those distributions are generated from the setup.py file, and > seems to be a convention used by PyPI. > @i...@ververica.com <i...@ververica.com> can you confirm here? > > On Sun, Mar 29, 2020 at 11:36 PM Hequn Cheng <he...@apache.org> wrote: > >> Hi Gordon, >> >> Thanks a lot for the new RC. I found some new blockers about licenses: >> >> - Module statefun-flink-distribution >> com.google.protobuf:protobuf-java:3.8.0 (The version should be 3.7.1) >> >> - Module statefun-ridesharing-example-simulator >> com.google.code.findbugs:jsr305:3.0.2:compile (Remove compile) >> org.hibernate.validator:hibernate-validator:6.0.17 (Version should be >> 6.0.17.Final) >> org.jboss.logging:jboss-logging:3.3.2 (Version should be 3.3.2.Final) >> >> Non-blocker feedback: >> - py3 is added in the name of "whl dist" but it is missing in the "source >> dist"[1]. Should we make them consistent? >> >> Best, >> Hequn >> >> [1] >> https://dist.apache.org/repos/dist/dev/flink/flink-statefun-2.0.0-rc3/ >> >> >> On Sun, Mar 29, 2020 at 3:31 PM Tzu-Li (Gordon) Tai <tzuli...@apache.org> >> wrote: >> >> > Hi everyone, >> > >> > Please review and vote on the *release candidate #3* for the version >> 2.0.0 >> > of Apache Flink Stateful Functions, >> > as follows: >> > [ ] +1, Approve the release >> > [ ] -1, Do not approve the release (please provide specific comments) >> > >> > **Testing Guideline** >> > >> > You can find here [1] a doc that we can use for collaborating testing >> > efforts. >> > The listed testing tasks in the doc also serve as a guideline in what to >> > test for this release. >> > If you wish to take ownership of a testing task, simply put your name >> down >> > in the "Checked by" field of the task. >> > >> > **Release Overview** >> > >> > As an overview, the release consists of the following: >> > a) Stateful Functions canonical source distribution, to be deployed to >> the >> > release repository at dist.apache.org >> > b) Stateful Functions Python SDK distributions to be deployed to PyPI >> > c) Maven artifacts to be deployed to the Maven Central Repository >> > >> > **Staging Areas to Review** >> > >> > The staging areas containing the above mentioned artifacts are as >> follows, >> > for your review: >> > * All artifacts for a) and b) can be found in the corresponding dev >> > repository at dist.apache.org [2] >> > * All artifacts for c) can be found at the Apache Nexus Repository [3] >> > >> > All artifacts are singed with the >> > key 1C1E2394D3194E1944613488F320986D35C33D6A [4] >> > >> > Other links for your review: >> > * JIRA release notes [5] >> > * source code tag "release-2.0.0-rc3" [6] [7] >> > >> > **Extra Remarks** >> > >> > * Part of the release is also official Docker images for Stateful >> > Functions. This can be a separate process, since the creation of those >> > relies on the fact that we have distribution jars already deployed to >> > Maven. I will follow-up with this after these artifacts are officially >> > released. >> > In the meantime, there is this discussion [8] ongoing about where to >> host >> > the StateFun Dockerfiles. >> > * The Flink Website and blog post is also being worked on (by Marta) as >> > part of the release, to incorporate the new Stateful Functions project. >> We >> > can follow up with a link to those changes afterwards in this vote >> thread, >> > but that would not block you to test and cast your votes already. >> > * Since the Flink website changes are still being worked on, you will >> not >> > yet be able to find the Stateful Functions docs from there. Here are the >> > links [9] [10]. >> > >> > **Vote Duration** >> > >> > The vote will be open for at least 72 hours starting Monday >> > *(target end date is Wednesday, April 1st).* >> > It is adopted by majority approval, with at least 3 PMC affirmative >> votes. >> > >> > Thanks, >> > Gordon >> > >> > [1] >> > >> > >> https://docs.google.com/document/d/1P9yjwSbPQtul0z2AXMnVolWQbzhxs68suJvzR6xMjcs/edit?usp=sharing >> > [2] >> https://dist.apache.org/repos/dist/dev/flink/flink-statefun-2.0.0-rc3/ >> > [3] >> > https://repository.apache.org/content/repositories/orgapacheflink-1342/ >> > [4] https://dist.apache.org/repos/dist/release/flink/KEYS >> > [5] >> > >> > >> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12346878 >> > [6] >> > >> > >> https://gitbox.apache.org/repos/asf?p=flink-statefun.git;a=commit;h=752e07fd9987ee430eb9d1c1d3fadff632ef9213 >> > [7] https://github.com/apache/flink-statefun/tree/release-2.0.0-rc3 >> > [8] >> > >> > >> http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Creating-a-new-repo-to-host-Stateful-Functions-Dockerfiles-td39342.html >> > [9] https://ci.apache.org/projects/flink/flink-statefun-docs-master/ >> > [10] >> https://ci.apache.org/projects/flink/flink-statefun-docs-release-2.0/ >> > >> > TIP: You can create a `settings.xml` file with these contents: >> > >> > """ >> > <settings> >> > <activeProfiles> >> > <activeProfile>flink-statefun-2.0.0</activeProfile> >> > </activeProfiles> >> > <profiles> >> > <profile> >> > <id>flink-statefun-2.0.0</id> >> > <repositories> >> > <repository> >> > <id>flink-statefun-2.0.0</id> >> > <url> >> > https://repository.apache.org/content/repositories/orgapacheflink-1342/ >> > </url> >> > </repository> >> > <repository> >> > <id>archetype</id> >> > <url> >> > https://repository.apache.org/content/repositories/orgapacheflink-1342/ >> > </url> >> > </repository> >> > </repositories> >> > </profile> >> > </profiles> >> > </settings> >> > """ >> > >> > And reference that in you maven commands via `--settings >> > path/to/settings.xml`. >> > This is useful for creating a quickstart based on the staged release and >> > for building against the staged jars. >> > >> >