+1 (binding) - checked the staged maven artifacts
On Tue, Apr 22, 2025 at 11:06 PM Sergey Nuyanzin <snuyan...@gmail.com> wrote: > +1 (binding) > > - verified signatures and hashsums > - checked RNs > - compiled from source > > one finding: > it seems KafkaTableITCase generates test artifacts not in a temporary > folder and it appears in git diff after that > I create a jira issue for that[1] > Based on git history the issue is present since Sep of 2023, so I > don't think it is a blocker > > [1] https://issues.apache.org/jira/browse/FLINK-37714 > > On Tue, Apr 22, 2025 at 10:14 AM weijie guo <guoweijieres...@gmail.com> > wrote: > > > > +1(binding) > > > > - verified signatures and hashsums > > - checked release tag > > - checked release note > > - compiled from source > > > > Best regards, > > > > Weijie > > > > > > Yanquan Lv <decq12y...@gmail.com> 于2025年4月22日周二 10:24写道： > > > > > Gentle ping for more binding votes. > > > > > > Tom Cooper <c...@tomcooper.dev> 于2025年4月17日周四 20:09写道： > > > > > > > Hi Arvid, > > > > > > > > Thanks for looking at this. I can confirm that updated sources now > > > compile > > > > on my Fedora box. I still get the following warning when running tar > -xf: > > > > > > > > tar: Ignoring unknown extended header keyword > > > > 'LIBARCHIVE.xattr.com.apple.provenance' > > > > > > > > Which I think is due to using BSD tar vs GNU. However, this isn't a > > > > blocker as far as I am concerned. > > > > > > > > I went back to the rest of the original RC3: > > > > > > > > - I reviewed the release notes > > > > - I verified the hashes > > > > - Build the source with Maven 3.9.9 and Java 11 + 17 > > > > - I ran tests using the staged connector jar as part of a custom > build of > > > > our [1] Flink SQL Runner (using minikube + Flink K8s Operator > 1.11.0) and > > > > verified it worked with Flink 1.20.1 and Kafka 3.9.0. > > > > > > > > So I am happy to change my -1 to a +1 (non-binding). > > > > > > > > Thanks, > > > > > > > > Tom Cooper > > > > @tomcooper.dev | https://tomcooper.dev > > > > > > > > [1] https://github.com/streamshub/flink-sql > > > > > > > > On Wednesday, 16 April 2025 at 22:36, Arvid Heise > > > > <ahe...@confluent.io.INVALID> wrote: > > > > > > > > > Hi Tom, > > > > > > > > > > thanks for catching that. I also released kafka-3.3 so that's odd. > I > > > > > released 3.3 and 3.4 within a couple of weeks. I know that Mac OS > likes > > > > to > > > > > create files that start with . like .DS_STORE which is used to > capture > > > > > thumbnails. I don't specifically know what has changed. > > > > > > > > > > In this case, it's about extended attributes of Mac that are > > > potentially > > > > > conveyed through dot underscore files. These are metadata files and > > > > > shouldn't be validated at all. Interestingly, I don't even see the > > > files > > > > in > > > > > mac when I use jar -tf or jar xf (while I do see .DS_STORE). These > > > files > > > > > also seem to be created only when I actually execute tar - I > couldn't > > > > make > > > > > them show up with any trick before. I unpacked the tar in Windows > and I > > > > see > > > > > basically such a file for each other file. > > > > > > > > > > So I see two issues that are wrong in our tooling: > > > > > * Files that start with . shouldn't be included in archives. > > > > > * Files that start with . shouldn't be checkstyled. > > > > > > > > > > I've fixed the first part now and reuploaded the source files (it > was > > > > > really painful to get them out during the stage process). Same > location > > > > as > > > > > before [1]. If sources are confirmed to look better, I'd merge the > > > > adjusted > > > > > scripts in our tooling repo. > > > > > > > > > > I'd leave it to Tom Cooper (or the broader community) to decide > whether > > > > we > > > > > need an RC4 or can just continue with RC3. Since none of the > sources > > > > needed > > > > > to be changed and the rc3 tag remains valid, I'd probably stick to > the > > > > > current RC but extend the voting period until Friday. However, I'd > > > > > explicitly need Tom to withdraw his -1 (and hopefully change it to > a > > > +1). > > > > > > > > > > Best, > > > > > > > > > > Arvid > > > > > [1] > > > > > > > > > > > > > https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-4.0.0-rc3 > > > > > > > > > > On Mon, Apr 14, 2025 at 6:03 PM Tom Cooper c...@tomcooper.dev > wrote: > > > > > > > > > > > Hi Martin, > > > > > > > > > > > > The source code build also fails on the 3.4 but not on the 3.3 > or 3.2 > > > > > > releases (I didn't go further back). It also does not fail on the > > > last > > > > two > > > > > > main Flink releases. So it seems that something changed in the > > > process > > > > for > > > > > > last two Kafka connector releases? > > > > > > > > > > > > It looks like Arvid Heise was the release manager for the 3.4 and > > > this > > > > 4.0 > > > > > > release. So maybe it is a config issue on their dev machine? > > > > > > > > > > > > From some light digging it seems the error messages we saw can be > > > > caused > > > > > > by using the native Apple BSD tools rather than the GNU version? > > > > > > > > > > > > I know this is probably not a priority issue (particularly as no > one > > > > seems > > > > > > to have complained about the 3.4 release), but it would be good > to > > > make > > > > > > sure the sources are usable by all. > > > > > > > > > > > > Tom Cooper > > > > > > @tomcooper.dev | https://tomcooper.dev > > > > > > > > > > > > On Monday, 14 April 2025 at 16:36, Martijn Visser < > > > > > > martijnvis...@apache.org> wrote: > > > > > > > > > > > > > Hi Tom, > > > > > > > > > > > > > > Can you check this on one of the approved releases too, given > that > > > > this > > > > > > > checkstyle issue is 10 years old and listed as a won't fix? > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > Martijn > > > > > > > > > > > > > > On Mon, Apr 14, 2025 at 3:35 PM Tom Cooper c...@tomcooper.dev > > > wrote: > > > > > > > > > > > > > > > -1 (non-binding) > > > > > > > > > > > > > > > > I have tested the RC3, however I have found an issue with the > > > > source > > > > > > > > code > > > > > > > > release. It seems that the checkstyle plugin is failing the > build > > > > (on > > > > > > > > my > > > > > > > > Fedora 40 Linux box with Maven 3.8.6 and Java 11/17) due to > the > > > > > > > > following > > > > > > > > error: > > > > > > > > > > > > > > > > [ERROR] Failed to execute goal > > > > > > > > org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check > > > > > > > > (validate) on > > > > > > > > project flink-connector-kafka: Failed during checkstyle > > > > configuration: > > > > > > > > Exception was thrown while processing > > > > > > > > > > > > > > > > > > > > /home/tcooper/Downloads/flink-connector-kafka-4.0.0/flink-connector-kafka/src/main/java/org/apache/flink/streaming/connectors/kafka/config/._StartupMode.java: > > > > > > > > > > > > > > TokenStreamRecognitionException occurred while parsing file > > > > > > > > > > > > > > > > > > > > /home/tcooper/Downloads/flink-connector-kafka-4.0.0/flink-connector-kafka/src/main/java/org/apache/flink/streaming/connectors/kafka/config/._StartupMode.java. > > > > > > > > > > > > > > Unexpected character 0x0 in identifier -> [Help 1] > > > > > > > > > > > > > > > > This error appears to be linked to the use of non-ASCII > > > characters. > > > > > > > > This > > > > > > > > appears to be a know limitation with checkstyle [1] that > effects > > > > > > > > non-Mac OS > > > > > > > > (which I assume is why no one else has reported the issue). > > > > > > > > > > > > > > > > I checked out the release tag (v4.0.0-rc3) and ran the build > from > > > > > > > > there, > > > > > > > > which passed without issues. So the issue is not with the > source. > > > > > > > > > > > > > > > > However, a possible clue came when I initially unpacked the > tar > > > > file. > > > > > > > > Using tar -xf on the source tar repeatedly printed the > following > > > > > > > > warning: > > > > > > > > > > > > > > > > tar: Ignoring unknown extended header keyword > > > > > > > > 'LIBARCHIVE.xattr.com.apple.provenance' > > > > > > > > > > > > > > > > Is it possible that whatever tooling you used to create the > > > source > > > > code > > > > > > > > tar file has injected characters into the files? > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > Tom Cooper > > > > > > > > @tomcooper.dev | https://tomcooper.dev > > > > > > > > > > > > > > > > [1] https://github.com/checkstyle/checkstyle/issues/2189 > > > > > > > > > > > > > > > > On Friday, 11 April 2025 at 10:55, Arvid Heise > > > > > > > > ahe...@confluent.io.INVALID > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Hi everyone, > > > > > > > > > Please review and vote on release candidate #3 for > > > > > > > > > flink-connector-kafka > > > > > > > > > v4.0.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], > > > > > > > > > which are signed with the key with fingerprint > 538B49E9BCF0B72F > > > > [3], > > > > > > > > > * all artifacts to be deployed to the Maven Central > Repository > > > > [4], > > > > > > > > > * source code tag v4.0.0-rc3 [5], > > > > > > > > > * website pull request listing the new release [6]. > > > > > > > > > * CI build of the tag [7]. > > > > > > > > > > > > > > > > > > The vote will be open for at least 72 hours. It is adopted > by > > > > > > > > > majority > > > > > > > > > approval, with at least 3 PMC affirmative votes. > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > Arvid > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352917 > > > > > > > > > > > > > > > [2] > > > > > > > > > > > > > > > > > > > > https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-4.0.0-rc3 > > > > > > > > > > > > > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS > > > > > > > > > [4] > > > > > > > > > > > > > > > > > > > > https://repository.apache.org/content/repositories/staging/org/apache/flink/flink-connector-kafka/4.0.0-2.0/ > > > > > > > > > > > > > > > [5] > > > > > > > > > > > > > > > > > https://github.com/apache/flink-connector-kafka/releases/tag/v4.0.0-rc3 > > > > > > > > > > > > > > > [6] https://github.com/apache/flink-web/pull/787 > > > > > > > > > [7] > > > > > > > > > > > > > > > > > > > > https://github.com/apache/flink-connector-kafka/actions/runs/14398219463/ > > > > > > > > > > > -- > Best regards, > Sergey >
