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/ >
