+1 (binding)

- built from source code succeeded
- verified signatures
- verified hashsums 
- checked Github release tag
- checked release notes
- reviewed the web PR 
- found the JDBC docs contains a valid link, reopened FLINK-31759[1] to fix it, 
not a blocker

Best,
Leonard
[1]https://issues.apache.org/jira/browse/FLINK-31759 
<https://issues.apache.org/jira/browse/FLINK-31759>

> 2023年5月2日 下午8:25,Danny Cranmer <dannycran...@apache.org> 写道:
> 
> +1 (binding)
> 
> - Release notes look good
> - Source archive signature and checksums good
> - Verified there are no binaries in the source archive
> - Contents of Maven repo look good
> - Built from source/tests pass
> - Tag is present in Github
> - Fixed web PR as per feedback
> 
> Thanks,
> Danny
> 
> On Wed, Apr 19, 2023 at 8:22 PM Martijn Visser <martijnvis...@apache.org>
> wrote:
> 
>> +1 (binding)
>> 
>> - Validated hashes
>> - Verified signature
>> - Verified that no binaries exist in the source archive
>> - Build the source with Maven
>> - Verified licenses
>> - Verified web PRs
>> 
>> On Wed, Apr 19, 2023 at 9:35 AM Martijn Visser <martijnvis...@apache.org>
>> wrote:
>> 
>>> Hi all,
>>> 
>>> There was already a blocker ticken op for this, that's
>>> https://issues.apache.org/jira/browse/FLINK-31770. There's also an open
>>> PR that should resolve it, but is waiting on a reviewer. See
>>> https://github.com/apache/flink-connector-jdbc/pull/22
>>> 
>>> I've already done the administration, so this is just fyi :)
>>> 
>>> Best regards,
>>> 
>>> Martijn
>>> 
>>> On Wed, Apr 19, 2023 at 9:17 AM Danny Cranmer <dannycran...@apache.org>
>>> wrote:
>>> 
>>>> Thank Benchao for the hotfix. I noticed the CI is also failing/flaky for
>>>> the main branch, I have raised a follow-up [1].
>>>> 
>>>> Danny,
>>>> 
>>>> [1] https://issues.apache.org/jira/browse/FLINK-31847
>>>> 
>>>> On Sun, Apr 16, 2023 at 10:57 AM Benchao Li <libenc...@apache.org>
>> wrote:
>>>> 
>>>>> And I've filed a hotfix PR[1] to fix the copyright issue.
>>>>> 
>>>>> [1] https://github.com/apache/flink-connector-jdbc/pull/41
>>>>> 
>>>>> Benchao Li <libenc...@apache.org> 于2023年4月16日周日 17:51写道:
>>>>> 
>>>>>> * Verified hashes, signatures (OK)
>>>>>> * Download, build and test source code (OK)
>>>>>> * Checked files in Nexus staging (OK)
>>>>>> * Start a Flink local cluster (1.17.0), run sql-client with jdbc
>>>>>> connector jar with Mysql 8.0.28 (OK)
>>>>>> * Go through the release note (OK)
>>>>>> 
>>>>>> One thing I spotted is that when I checked the NOTICE file, I found
>>>> that
>>>>>> we have not updated the copyright year to 2023, I'm not sure whether
>>>> we
>>>>>> should take this as a blocker issue. If not, my vote is +1.
>>>>>> 
>>>>>> And I left my comments in the PR for website updating.
>>>>>> 
>>>>>> Teoh, Hong <lian...@amazon.co.uk.invalid> 于2023年4月14日周五 15:54写道:
>>>>>> 
>>>>>>> Thanks Danny
>>>>>>> 
>>>>>>> +1 (non-binding)
>>>>>>> 
>>>>>>> * Hashes and Signatures look good
>>>>>>> * All required files on dist.apache.org
>>>>>>> * Tag is present in Github
>>>>>>> * Verified source archive does not contain any binary files
>>>>>>> * Source archive builds using maven
>>>>>>> 
>>>>>>> 
>>>>>>> Cheers,
>>>>>>> Hong
>>>>>>> 
>>>>>>> 
>>>>>>>> On 14 Apr 2023, at 05:21, Elphas Toringepi <elphast...@gmail.com
>>> 
>>>>>>> wrote:
>>>>>>>> 
>>>>>>>> CAUTION: This email originated from outside of the organization.
>> Do
>>>>> not
>>>>>>> click links or open attachments unless you can confirm the sender
>> and
>>>>> know
>>>>>>> the content is safe.
>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>> +1 (non-binding)
>>>>>>>> 
>>>>>>>> * JIRA release notes look good
>>>>>>>> * Verified signature and checksum for Apache source
>>>>>>>> * Checked source code tag and confirmed Github and Apache source
>>>>>>> releases
>>>>>>>> are identical
>>>>>>>> * Reviewed website pull request and left comment
>>>>>>>> 
>>>>>>>> Kind regards,
>>>>>>>> Elphas
>>>>>>>> 
>>>>>>>> 
>>>>>>>> On Thu, Apr 13, 2023 at 3:42 PM Danny Cranmer <
>>>>> dannycran...@apache.org>
>>>>>>>> wrote:
>>>>>>>> 
>>>>>>>>> Hi everyone,
>>>>>>>>> Please review and vote on the release candidate #1 for the
>> version
>>>>>>> v3.1.0,
>>>>>>>>> as follows:
>>>>>>>>> [ ] +1, Approve the release
>>>>>>>>> [ ] -1, Do not approve the release (please provide specific
>>>> comments)
>>>>>>>>> 
>>>>>>>>> This version supports both Flink 1.16.x and Flink 1.17.x
>>>>>>>>> 
>>>>>>>>> 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
>>>>>>>>> 0F79F2AFB2351BC29678544591F9C1EC125FD8DB [3],
>>>>>>>>> * all artifacts to be deployed to the Maven Central Repository
>>>> [4],
>>>>>>>>> * source code tag v3.1.0-rc1 [5],
>>>>>>>>> * website pull request listing the new release [6].
>>>>>>>>> 
>>>>>>>>> The vote will be open for at least 72 hours. It is adopted by
>>>>> majority
>>>>>>>>> approval, with at least 3 PMC affirmative votes.
>>>>>>>>> 
>>>>>>>>> Thanks,
>>>>>>>>> Danny
>>>>>>>>> 
>>>>>>>>> [1]
>>>>>>>>> 
>>>>>>>>> 
>>>>>>> 
>>>>> 
>>>> 
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352678
>>>>>>>>> [2]
>>>>>>>>> 
>>>>>>> 
>>>>> 
>>>> 
>> https://dist.apache.org/repos/dist/dev/flink/flink-connector-jdbc-3.1.0-rc1
>>>>>>>>> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
>>>>>>>>> [4]
>>>>>>>>> 
>>>>>>> 
>>>> https://repository.apache.org/content/repositories/orgapacheflink-1614/
>>>>>>>>> [5]
>>>>>>> 
>>>> https://github.com/apache/flink-connector-jdbc/releases/tag/v3.1.0-rc1
>>>>>>>>> [6] https://github.com/apache/flink-web/pull/638
>>>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>> 
>>>>>> --
>>>>>> 
>>>>>> Best,
>>>>>> Benchao Li
>>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> 
>>>>> Best,
>>>>> Benchao Li
>>>>> 
>>>> 
>>> 
>> 

Reply via email to