Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-12 Thread Tzu-Li (Gordon) Tai
RC2 (for Flink 1.17.0) vote has started in a separate thread:
https://lists.apache.org/thread/mff76c2hzcb1mk8fm5m2h4z0j73qz2vk

Please test and cast your votes!

On Tue, Apr 11, 2023 at 11:45 AM Martijn Visser 
wrote:

> +1, thanks for driving this Gordon.
>
> On Tue, Apr 11, 2023 at 8:15 PM Tzu-Li (Gordon) Tai 
> wrote:
>
>> Hi all,
>>
>> Martijn and I discussed offline to cancel this vote.
>>
>> Moreover, now that Flink 1.17 is out and we still haven't released
>> anything yet for the newly externalized Kafka connector, we've decided to
>> skip releasing a version that matches with Flink 1.16 all together, and
>> instead go straight to supporting Flink 1.17 for our first release.
>>
>> Practically this means:
>>
>>1. The code as of branch `flink-connector-kafka:v4.0` will be
>>re-versioned as `v3.0` and that will be the actual first release of
>>flink-connector-kafka.
>>2. v3.0.0 will be the first release of `flink-connector-kafka` and it
>>will initially support Flink 1.17.x series.
>>
>> I'm happy to drive the release efforts for this and will create a new RC
>> shortly over the next day or two.
>>
>> Thanks,
>> Gordon
>>
>> On Wed, Apr 5, 2023 at 9:32 PM Mason Chen  wrote:
>>
>>> +1 for new RC!
>>>
>>> Best,
>>> Mason
>>>
>>> On Tue, Apr 4, 2023 at 11:32 AM Tzu-Li (Gordon) Tai >> >
>>> wrote:
>>>
>>> > Hi all,
>>> >
>>> > I've ported the critical fixes I mentioned to v3.0 and v4.0 branches of
>>> > apache/flink-connector-kafka now.
>>> >
>>> > @martijnvis...@apache.org  let me know if
>>> you'd
>>> > need help with creating a new RC, if there's too much to juggle on
>>> > your end. Happy to help out.
>>> >
>>> > Thanks,
>>> > Gordon
>>> >
>>> > On Sun, Apr 2, 2023 at 11:21 PM Konstantin Knauf 
>>> > wrote:
>>> >
>>> > > +1. Thanks, Gordon!
>>> > >
>>> > > Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
>>> > > tzuli...@apache.org>:
>>> > >
>>> > > > Hi Martijn,
>>> > > >
>>> > > > Since this RC vote was opened, we had three critical bug fixes
>>> that was
>>> > > > merged for the Kafka connector:
>>> > > >
>>> > > >- https://issues.apache.org/jira/browse/FLINK-31363
>>> > > >- https://issues.apache.org/jira/browse/FLINK-31305
>>> > > >- https://issues.apache.org/jira/browse/FLINK-31620
>>> > > >
>>> > > > Given the severity of these issues (all of them are violations of
>>> > > > exactly-once semantics), and the fact that they are currently not
>>> > > included
>>> > > > yet in any released version, do you think it makes sense to cancel
>>> this
>>> > > RC
>>> > > > in favor of a new one that includes these?
>>> > > > Since this RC vote has been stale for quite some time already, it
>>> > doesn't
>>> > > > seem like we're throwing away too much effort that has already been
>>> > done
>>> > > if
>>> > > > we start a new RC with these critical fixes included.
>>> > > >
>>> > > > What do you think?
>>> > > >
>>> > > > Thanks,
>>> > > > Gordon
>>> > > >
>>> > > > On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai <
>>> > tzuli...@apache.org>
>>> > > > wrote:
>>> > > >
>>> > > > > +1 (binding)
>>> > > > >
>>> > > > > - Verified legals (license headers and root LICENSE / NOTICE
>>> file).
>>> > > > AFAICT
>>> > > > > no dependencies require explicit acknowledgement in the NOTICE
>>> files.
>>> > > > > - No binaries in staging area
>>> > > > > - Built source with tests
>>> > > > > - Verified signatures and hashes
>>> > > > > - Web PR changes LGTM
>>> > > > >
>>> > > > > Thanks Martijn!
>>> > > > >
>>> > > > > Cheers,
>>> > > > > Gordon
>>> > > > >
>>> > > > > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen <
>>> mas.chen6...@gmail.com>
>>> > > > wrote:
>>> > > > >
>>> > > > >> That makes sense, thanks for the clarification!
>>> > > > >>
>>> > > > >> Best,
>>> > > > >> Mason
>>> > > > >>
>>> > > > >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser <
>>> > > martijnvis...@apache.org
>>> > > > >
>>> > > > >> wrote:
>>> > > > >>
>>> > > > >> > Hi Mason,
>>> > > > >> >
>>> > > > >> > Thanks, [4] is indeed a copy-paste error and you've made the
>>> right
>>> > > > >> > assumption that
>>> > > > >> >
>>> > > > >> >
>>> > > > >>
>>> > > >
>>> > >
>>> >
>>> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
>>> > > > >> > is the correct maven central link.
>>> > > > >> >
>>> > > > >> > I think we should use FLINK-30052 to move the Kafka connector
>>> code
>>> > > > from
>>> > > > >> the
>>> > > > >> > 1.17 release also over the Kafka connector repo (especially
>>> since
>>> > > > >> there's
>>> > > > >> > now a v3.0 branch for the Kafka connector, so it can be
>>> merged in
>>> > > > main).
>>> > > > >> > When those commits have been merged, we can make a next Kafka
>>> > > > connector
>>> > > > >> > release (which is equivalent to the 1.17 release, which can
>>> only
>>> > be
>>> > > > done
>>> > > > >> > when 1.17 is done because of the split level watermark
>>> alignment)
>>> > > and
>>> > > > >> then
>>> > > > >> > FLINK-30859 can be fi

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-11 Thread Martijn Visser
+1, thanks for driving this Gordon.

On Tue, Apr 11, 2023 at 8:15 PM Tzu-Li (Gordon) Tai 
wrote:

> Hi all,
>
> Martijn and I discussed offline to cancel this vote.
>
> Moreover, now that Flink 1.17 is out and we still haven't released
> anything yet for the newly externalized Kafka connector, we've decided to
> skip releasing a version that matches with Flink 1.16 all together, and
> instead go straight to supporting Flink 1.17 for our first release.
>
> Practically this means:
>
>1. The code as of branch `flink-connector-kafka:v4.0` will be
>re-versioned as `v3.0` and that will be the actual first release of
>flink-connector-kafka.
>2. v3.0.0 will be the first release of `flink-connector-kafka` and it
>will initially support Flink 1.17.x series.
>
> I'm happy to drive the release efforts for this and will create a new RC
> shortly over the next day or two.
>
> Thanks,
> Gordon
>
> On Wed, Apr 5, 2023 at 9:32 PM Mason Chen  wrote:
>
>> +1 for new RC!
>>
>> Best,
>> Mason
>>
>> On Tue, Apr 4, 2023 at 11:32 AM Tzu-Li (Gordon) Tai 
>> wrote:
>>
>> > Hi all,
>> >
>> > I've ported the critical fixes I mentioned to v3.0 and v4.0 branches of
>> > apache/flink-connector-kafka now.
>> >
>> > @martijnvis...@apache.org  let me know if
>> you'd
>> > need help with creating a new RC, if there's too much to juggle on
>> > your end. Happy to help out.
>> >
>> > Thanks,
>> > Gordon
>> >
>> > On Sun, Apr 2, 2023 at 11:21 PM Konstantin Knauf 
>> > wrote:
>> >
>> > > +1. Thanks, Gordon!
>> > >
>> > > Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
>> > > tzuli...@apache.org>:
>> > >
>> > > > Hi Martijn,
>> > > >
>> > > > Since this RC vote was opened, we had three critical bug fixes that
>> was
>> > > > merged for the Kafka connector:
>> > > >
>> > > >- https://issues.apache.org/jira/browse/FLINK-31363
>> > > >- https://issues.apache.org/jira/browse/FLINK-31305
>> > > >- https://issues.apache.org/jira/browse/FLINK-31620
>> > > >
>> > > > Given the severity of these issues (all of them are violations of
>> > > > exactly-once semantics), and the fact that they are currently not
>> > > included
>> > > > yet in any released version, do you think it makes sense to cancel
>> this
>> > > RC
>> > > > in favor of a new one that includes these?
>> > > > Since this RC vote has been stale for quite some time already, it
>> > doesn't
>> > > > seem like we're throwing away too much effort that has already been
>> > done
>> > > if
>> > > > we start a new RC with these critical fixes included.
>> > > >
>> > > > What do you think?
>> > > >
>> > > > Thanks,
>> > > > Gordon
>> > > >
>> > > > On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai <
>> > tzuli...@apache.org>
>> > > > wrote:
>> > > >
>> > > > > +1 (binding)
>> > > > >
>> > > > > - Verified legals (license headers and root LICENSE / NOTICE
>> file).
>> > > > AFAICT
>> > > > > no dependencies require explicit acknowledgement in the NOTICE
>> files.
>> > > > > - No binaries in staging area
>> > > > > - Built source with tests
>> > > > > - Verified signatures and hashes
>> > > > > - Web PR changes LGTM
>> > > > >
>> > > > > Thanks Martijn!
>> > > > >
>> > > > > Cheers,
>> > > > > Gordon
>> > > > >
>> > > > > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen > >
>> > > > wrote:
>> > > > >
>> > > > >> That makes sense, thanks for the clarification!
>> > > > >>
>> > > > >> Best,
>> > > > >> Mason
>> > > > >>
>> > > > >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser <
>> > > martijnvis...@apache.org
>> > > > >
>> > > > >> wrote:
>> > > > >>
>> > > > >> > Hi Mason,
>> > > > >> >
>> > > > >> > Thanks, [4] is indeed a copy-paste error and you've made the
>> right
>> > > > >> > assumption that
>> > > > >> >
>> > > > >> >
>> > > > >>
>> > > >
>> > >
>> >
>> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
>> > > > >> > is the correct maven central link.
>> > > > >> >
>> > > > >> > I think we should use FLINK-30052 to move the Kafka connector
>> code
>> > > > from
>> > > > >> the
>> > > > >> > 1.17 release also over the Kafka connector repo (especially
>> since
>> > > > >> there's
>> > > > >> > now a v3.0 branch for the Kafka connector, so it can be merged
>> in
>> > > > main).
>> > > > >> > When those commits have been merged, we can make a next Kafka
>> > > > connector
>> > > > >> > release (which is equivalent to the 1.17 release, which can
>> only
>> > be
>> > > > done
>> > > > >> > when 1.17 is done because of the split level watermark
>> alignment)
>> > > and
>> > > > >> then
>> > > > >> > FLINK-30859 can be finished.
>> > > > >> >
>> > > > >> > Best regards,
>> > > > >> >
>> > > > >> > Martijn
>> > > > >> >
>> > > > >> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen <
>> > > mas.chen6...@gmail.com
>> > > > >:
>> > > > >> >
>> > > > >> > > +1 (non-binding)
>> > > > >> > >
>> > > > >> > > * Verified hashes and signatures
>> > > > >> > > * Verified no binaries
>> > > > >> > > * Verified LICENSE and NOTICE files
>>

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-11 Thread Tzu-Li (Gordon) Tai
Hi all,

Martijn and I discussed offline to cancel this vote.

Moreover, now that Flink 1.17 is out and we still haven't released anything
yet for the newly externalized Kafka connector, we've decided to skip
releasing a version that matches with Flink 1.16 all together, and instead
go straight to supporting Flink 1.17 for our first release.

Practically this means:

   1. The code as of branch `flink-connector-kafka:v4.0` will be
   re-versioned as `v3.0` and that will be the actual first release of
   flink-connector-kafka.
   2. v3.0.0 will be the first release of `flink-connector-kafka` and it
   will initially support Flink 1.17.x series.

I'm happy to drive the release efforts for this and will create a new RC
shortly over the next day or two.

Thanks,
Gordon

On Wed, Apr 5, 2023 at 9:32 PM Mason Chen  wrote:

> +1 for new RC!
>
> Best,
> Mason
>
> On Tue, Apr 4, 2023 at 11:32 AM Tzu-Li (Gordon) Tai 
> wrote:
>
> > Hi all,
> >
> > I've ported the critical fixes I mentioned to v3.0 and v4.0 branches of
> > apache/flink-connector-kafka now.
> >
> > @martijnvis...@apache.org  let me know if
> you'd
> > need help with creating a new RC, if there's too much to juggle on
> > your end. Happy to help out.
> >
> > Thanks,
> > Gordon
> >
> > On Sun, Apr 2, 2023 at 11:21 PM Konstantin Knauf 
> > wrote:
> >
> > > +1. Thanks, Gordon!
> > >
> > > Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
> > > tzuli...@apache.org>:
> > >
> > > > Hi Martijn,
> > > >
> > > > Since this RC vote was opened, we had three critical bug fixes that
> was
> > > > merged for the Kafka connector:
> > > >
> > > >- https://issues.apache.org/jira/browse/FLINK-31363
> > > >- https://issues.apache.org/jira/browse/FLINK-31305
> > > >- https://issues.apache.org/jira/browse/FLINK-31620
> > > >
> > > > Given the severity of these issues (all of them are violations of
> > > > exactly-once semantics), and the fact that they are currently not
> > > included
> > > > yet in any released version, do you think it makes sense to cancel
> this
> > > RC
> > > > in favor of a new one that includes these?
> > > > Since this RC vote has been stale for quite some time already, it
> > doesn't
> > > > seem like we're throwing away too much effort that has already been
> > done
> > > if
> > > > we start a new RC with these critical fixes included.
> > > >
> > > > What do you think?
> > > >
> > > > Thanks,
> > > > Gordon
> > > >
> > > > On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai <
> > tzuli...@apache.org>
> > > > wrote:
> > > >
> > > > > +1 (binding)
> > > > >
> > > > > - Verified legals (license headers and root LICENSE / NOTICE file).
> > > > AFAICT
> > > > > no dependencies require explicit acknowledgement in the NOTICE
> files.
> > > > > - No binaries in staging area
> > > > > - Built source with tests
> > > > > - Verified signatures and hashes
> > > > > - Web PR changes LGTM
> > > > >
> > > > > Thanks Martijn!
> > > > >
> > > > > Cheers,
> > > > > Gordon
> > > > >
> > > > > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen 
> > > > wrote:
> > > > >
> > > > >> That makes sense, thanks for the clarification!
> > > > >>
> > > > >> Best,
> > > > >> Mason
> > > > >>
> > > > >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser <
> > > martijnvis...@apache.org
> > > > >
> > > > >> wrote:
> > > > >>
> > > > >> > Hi Mason,
> > > > >> >
> > > > >> > Thanks, [4] is indeed a copy-paste error and you've made the
> right
> > > > >> > assumption that
> > > > >> >
> > > > >> >
> > > > >>
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > > > >> > is the correct maven central link.
> > > > >> >
> > > > >> > I think we should use FLINK-30052 to move the Kafka connector
> code
> > > > from
> > > > >> the
> > > > >> > 1.17 release also over the Kafka connector repo (especially
> since
> > > > >> there's
> > > > >> > now a v3.0 branch for the Kafka connector, so it can be merged
> in
> > > > main).
> > > > >> > When those commits have been merged, we can make a next Kafka
> > > > connector
> > > > >> > release (which is equivalent to the 1.17 release, which can only
> > be
> > > > done
> > > > >> > when 1.17 is done because of the split level watermark
> alignment)
> > > and
> > > > >> then
> > > > >> > FLINK-30859 can be finished.
> > > > >> >
> > > > >> > Best regards,
> > > > >> >
> > > > >> > Martijn
> > > > >> >
> > > > >> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen <
> > > mas.chen6...@gmail.com
> > > > >:
> > > > >> >
> > > > >> > > +1 (non-binding)
> > > > >> > >
> > > > >> > > * Verified hashes and signatures
> > > > >> > > * Verified no binaries
> > > > >> > > * Verified LICENSE and NOTICE files
> > > > >> > > * Verified poms point to 3.0.0-1.16
> > > > >> > > * Reviewed web PR
> > > > >> > > * Built from source
> > > > >> > > * Verified git tag
> > > > >> > >
> > > > >> > > I think [4] your is a copy-paste error and I did all the
> > > > verification
> > > > >> > > assuming that

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-05 Thread Mason Chen
+1 for new RC!

Best,
Mason

On Tue, Apr 4, 2023 at 11:32 AM Tzu-Li (Gordon) Tai 
wrote:

> Hi all,
>
> I've ported the critical fixes I mentioned to v3.0 and v4.0 branches of
> apache/flink-connector-kafka now.
>
> @martijnvis...@apache.org  let me know if you'd
> need help with creating a new RC, if there's too much to juggle on
> your end. Happy to help out.
>
> Thanks,
> Gordon
>
> On Sun, Apr 2, 2023 at 11:21 PM Konstantin Knauf 
> wrote:
>
> > +1. Thanks, Gordon!
> >
> > Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
> > tzuli...@apache.org>:
> >
> > > Hi Martijn,
> > >
> > > Since this RC vote was opened, we had three critical bug fixes that was
> > > merged for the Kafka connector:
> > >
> > >- https://issues.apache.org/jira/browse/FLINK-31363
> > >- https://issues.apache.org/jira/browse/FLINK-31305
> > >- https://issues.apache.org/jira/browse/FLINK-31620
> > >
> > > Given the severity of these issues (all of them are violations of
> > > exactly-once semantics), and the fact that they are currently not
> > included
> > > yet in any released version, do you think it makes sense to cancel this
> > RC
> > > in favor of a new one that includes these?
> > > Since this RC vote has been stale for quite some time already, it
> doesn't
> > > seem like we're throwing away too much effort that has already been
> done
> > if
> > > we start a new RC with these critical fixes included.
> > >
> > > What do you think?
> > >
> > > Thanks,
> > > Gordon
> > >
> > > On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai <
> tzuli...@apache.org>
> > > wrote:
> > >
> > > > +1 (binding)
> > > >
> > > > - Verified legals (license headers and root LICENSE / NOTICE file).
> > > AFAICT
> > > > no dependencies require explicit acknowledgement in the NOTICE files.
> > > > - No binaries in staging area
> > > > - Built source with tests
> > > > - Verified signatures and hashes
> > > > - Web PR changes LGTM
> > > >
> > > > Thanks Martijn!
> > > >
> > > > Cheers,
> > > > Gordon
> > > >
> > > > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen 
> > > wrote:
> > > >
> > > >> That makes sense, thanks for the clarification!
> > > >>
> > > >> Best,
> > > >> Mason
> > > >>
> > > >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser <
> > martijnvis...@apache.org
> > > >
> > > >> wrote:
> > > >>
> > > >> > Hi Mason,
> > > >> >
> > > >> > Thanks, [4] is indeed a copy-paste error and you've made the right
> > > >> > assumption that
> > > >> >
> > > >> >
> > > >>
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > > >> > is the correct maven central link.
> > > >> >
> > > >> > I think we should use FLINK-30052 to move the Kafka connector code
> > > from
> > > >> the
> > > >> > 1.17 release also over the Kafka connector repo (especially since
> > > >> there's
> > > >> > now a v3.0 branch for the Kafka connector, so it can be merged in
> > > main).
> > > >> > When those commits have been merged, we can make a next Kafka
> > > connector
> > > >> > release (which is equivalent to the 1.17 release, which can only
> be
> > > done
> > > >> > when 1.17 is done because of the split level watermark alignment)
> > and
> > > >> then
> > > >> > FLINK-30859 can be finished.
> > > >> >
> > > >> > Best regards,
> > > >> >
> > > >> > Martijn
> > > >> >
> > > >> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen <
> > mas.chen6...@gmail.com
> > > >:
> > > >> >
> > > >> > > +1 (non-binding)
> > > >> > >
> > > >> > > * Verified hashes and signatures
> > > >> > > * Verified no binaries
> > > >> > > * Verified LICENSE and NOTICE files
> > > >> > > * Verified poms point to 3.0.0-1.16
> > > >> > > * Reviewed web PR
> > > >> > > * Built from source
> > > >> > > * Verified git tag
> > > >> > >
> > > >> > > I think [4] your is a copy-paste error and I did all the
> > > verification
> > > >> > > assuming that
> > > >> > >
> > > >> > >
> > > >> >
> > > >>
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > > >> > > is the correct maven central link.
> > > >> > >
> > > >> > > Regarding the release notes, should we close
> > > >> > > https://issues.apache.org/jira/browse/FLINK-30052 and link it
> > > there?
> > > >> > I've
> > > >> > > created https://issues.apache.org/jira/browse/FLINK-30859 to
> > remove
> > > >> the
> > > >> > > existing code from the master branch.
> > > >> > >
> > > >> > > Best,
> > > >> > > Mason
> > > >> > >
> > > >> > > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser <
> > > >> martijnvis...@apache.org
> > > >> > >
> > > >> > > wrote:
> > > >> > >
> > > >> > > > Hi everyone,
> > > >> > > > Please review and vote on the release candidate #1 for
> > > >> > > > flink-connector-kafka version 3.0.0, as follows:
> > > >> > > > [ ] +1, Approve the release
> > > >> > > > [ ] -1, Do not approve the release (please provide specific
> > > >> comments)
> > > >> > > >
> > > >> > > > Note: this is the same code as the Kafka connector for the
> Flin

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-04 Thread Tzu-Li (Gordon) Tai
Hi all,

I've ported the critical fixes I mentioned to v3.0 and v4.0 branches of
apache/flink-connector-kafka now.

@martijnvis...@apache.org  let me know if you'd
need help with creating a new RC, if there's too much to juggle on
your end. Happy to help out.

Thanks,
Gordon

On Sun, Apr 2, 2023 at 11:21 PM Konstantin Knauf  wrote:

> +1. Thanks, Gordon!
>
> Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
> tzuli...@apache.org>:
>
> > Hi Martijn,
> >
> > Since this RC vote was opened, we had three critical bug fixes that was
> > merged for the Kafka connector:
> >
> >- https://issues.apache.org/jira/browse/FLINK-31363
> >- https://issues.apache.org/jira/browse/FLINK-31305
> >- https://issues.apache.org/jira/browse/FLINK-31620
> >
> > Given the severity of these issues (all of them are violations of
> > exactly-once semantics), and the fact that they are currently not
> included
> > yet in any released version, do you think it makes sense to cancel this
> RC
> > in favor of a new one that includes these?
> > Since this RC vote has been stale for quite some time already, it doesn't
> > seem like we're throwing away too much effort that has already been done
> if
> > we start a new RC with these critical fixes included.
> >
> > What do you think?
> >
> > Thanks,
> > Gordon
> >
> > On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai 
> > wrote:
> >
> > > +1 (binding)
> > >
> > > - Verified legals (license headers and root LICENSE / NOTICE file).
> > AFAICT
> > > no dependencies require explicit acknowledgement in the NOTICE files.
> > > - No binaries in staging area
> > > - Built source with tests
> > > - Verified signatures and hashes
> > > - Web PR changes LGTM
> > >
> > > Thanks Martijn!
> > >
> > > Cheers,
> > > Gordon
> > >
> > > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen 
> > wrote:
> > >
> > >> That makes sense, thanks for the clarification!
> > >>
> > >> Best,
> > >> Mason
> > >>
> > >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser <
> martijnvis...@apache.org
> > >
> > >> wrote:
> > >>
> > >> > Hi Mason,
> > >> >
> > >> > Thanks, [4] is indeed a copy-paste error and you've made the right
> > >> > assumption that
> > >> >
> > >> >
> > >>
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > >> > is the correct maven central link.
> > >> >
> > >> > I think we should use FLINK-30052 to move the Kafka connector code
> > from
> > >> the
> > >> > 1.17 release also over the Kafka connector repo (especially since
> > >> there's
> > >> > now a v3.0 branch for the Kafka connector, so it can be merged in
> > main).
> > >> > When those commits have been merged, we can make a next Kafka
> > connector
> > >> > release (which is equivalent to the 1.17 release, which can only be
> > done
> > >> > when 1.17 is done because of the split level watermark alignment)
> and
> > >> then
> > >> > FLINK-30859 can be finished.
> > >> >
> > >> > Best regards,
> > >> >
> > >> > Martijn
> > >> >
> > >> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen <
> mas.chen6...@gmail.com
> > >:
> > >> >
> > >> > > +1 (non-binding)
> > >> > >
> > >> > > * Verified hashes and signatures
> > >> > > * Verified no binaries
> > >> > > * Verified LICENSE and NOTICE files
> > >> > > * Verified poms point to 3.0.0-1.16
> > >> > > * Reviewed web PR
> > >> > > * Built from source
> > >> > > * Verified git tag
> > >> > >
> > >> > > I think [4] your is a copy-paste error and I did all the
> > verification
> > >> > > assuming that
> > >> > >
> > >> > >
> > >> >
> > >>
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > >> > > is the correct maven central link.
> > >> > >
> > >> > > Regarding the release notes, should we close
> > >> > > https://issues.apache.org/jira/browse/FLINK-30052 and link it
> > there?
> > >> > I've
> > >> > > created https://issues.apache.org/jira/browse/FLINK-30859 to
> remove
> > >> the
> > >> > > existing code from the master branch.
> > >> > >
> > >> > > Best,
> > >> > > Mason
> > >> > >
> > >> > > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser <
> > >> martijnvis...@apache.org
> > >> > >
> > >> > > wrote:
> > >> > >
> > >> > > > Hi everyone,
> > >> > > > Please review and vote on the release candidate #1 for
> > >> > > > flink-connector-kafka version 3.0.0, as follows:
> > >> > > > [ ] +1, Approve the release
> > >> > > > [ ] -1, Do not approve the release (please provide specific
> > >> comments)
> > >> > > >
> > >> > > > Note: this is the same code as the Kafka connector for the Flink
> > >> 1.16
> > >> > > > release.
> > >> > > >
> > >> > > > 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
> > >> > > > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> > >> > > > * all artifact

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-02 Thread Konstantin Knauf
+1. Thanks, Gordon!

Am Mo., 3. Apr. 2023 um 06:37 Uhr schrieb Tzu-Li (Gordon) Tai <
tzuli...@apache.org>:

> Hi Martijn,
>
> Since this RC vote was opened, we had three critical bug fixes that was
> merged for the Kafka connector:
>
>- https://issues.apache.org/jira/browse/FLINK-31363
>- https://issues.apache.org/jira/browse/FLINK-31305
>- https://issues.apache.org/jira/browse/FLINK-31620
>
> Given the severity of these issues (all of them are violations of
> exactly-once semantics), and the fact that they are currently not included
> yet in any released version, do you think it makes sense to cancel this RC
> in favor of a new one that includes these?
> Since this RC vote has been stale for quite some time already, it doesn't
> seem like we're throwing away too much effort that has already been done if
> we start a new RC with these critical fixes included.
>
> What do you think?
>
> Thanks,
> Gordon
>
> On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai 
> wrote:
>
> > +1 (binding)
> >
> > - Verified legals (license headers and root LICENSE / NOTICE file).
> AFAICT
> > no dependencies require explicit acknowledgement in the NOTICE files.
> > - No binaries in staging area
> > - Built source with tests
> > - Verified signatures and hashes
> > - Web PR changes LGTM
> >
> > Thanks Martijn!
> >
> > Cheers,
> > Gordon
> >
> > On Mon, Feb 6, 2023 at 6:12 PM Mason Chen 
> wrote:
> >
> >> That makes sense, thanks for the clarification!
> >>
> >> Best,
> >> Mason
> >>
> >> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser  >
> >> wrote:
> >>
> >> > Hi Mason,
> >> >
> >> > Thanks, [4] is indeed a copy-paste error and you've made the right
> >> > assumption that
> >> >
> >> >
> >>
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> >> > is the correct maven central link.
> >> >
> >> > I think we should use FLINK-30052 to move the Kafka connector code
> from
> >> the
> >> > 1.17 release also over the Kafka connector repo (especially since
> >> there's
> >> > now a v3.0 branch for the Kafka connector, so it can be merged in
> main).
> >> > When those commits have been merged, we can make a next Kafka
> connector
> >> > release (which is equivalent to the 1.17 release, which can only be
> done
> >> > when 1.17 is done because of the split level watermark alignment) and
> >> then
> >> > FLINK-30859 can be finished.
> >> >
> >> > Best regards,
> >> >
> >> > Martijn
> >> >
> >> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen  >:
> >> >
> >> > > +1 (non-binding)
> >> > >
> >> > > * Verified hashes and signatures
> >> > > * Verified no binaries
> >> > > * Verified LICENSE and NOTICE files
> >> > > * Verified poms point to 3.0.0-1.16
> >> > > * Reviewed web PR
> >> > > * Built from source
> >> > > * Verified git tag
> >> > >
> >> > > I think [4] your is a copy-paste error and I did all the
> verification
> >> > > assuming that
> >> > >
> >> > >
> >> >
> >>
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> >> > > is the correct maven central link.
> >> > >
> >> > > Regarding the release notes, should we close
> >> > > https://issues.apache.org/jira/browse/FLINK-30052 and link it
> there?
> >> > I've
> >> > > created https://issues.apache.org/jira/browse/FLINK-30859 to remove
> >> the
> >> > > existing code from the master branch.
> >> > >
> >> > > Best,
> >> > > Mason
> >> > >
> >> > > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser <
> >> martijnvis...@apache.org
> >> > >
> >> > > wrote:
> >> > >
> >> > > > Hi everyone,
> >> > > > Please review and vote on the release candidate #1 for
> >> > > > flink-connector-kafka version 3.0.0, as follows:
> >> > > > [ ] +1, Approve the release
> >> > > > [ ] -1, Do not approve the release (please provide specific
> >> comments)
> >> > > >
> >> > > > Note: this is the same code as the Kafka connector for the Flink
> >> 1.16
> >> > > > release.
> >> > > >
> >> > > > 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
> >> > > > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> >> > > > * all artifacts to be deployed to the Maven Central Repository
> [4],
> >> > > > * source code tag v3.0.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,
> >> > > > Release Manager
> >> > > >
> >> > > > [1]
> >> > > >
> >> > > >
> >> > >
> >> >
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
> >> > > > [2]
> >> > > >
> >> > > >
> >> > >
> >> >
> >>
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
> >> > > > [3] ht

Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-04-02 Thread Tzu-Li (Gordon) Tai
Hi Martijn,

Since this RC vote was opened, we had three critical bug fixes that was
merged for the Kafka connector:

   - https://issues.apache.org/jira/browse/FLINK-31363
   - https://issues.apache.org/jira/browse/FLINK-31305
   - https://issues.apache.org/jira/browse/FLINK-31620

Given the severity of these issues (all of them are violations of
exactly-once semantics), and the fact that they are currently not included
yet in any released version, do you think it makes sense to cancel this RC
in favor of a new one that includes these?
Since this RC vote has been stale for quite some time already, it doesn't
seem like we're throwing away too much effort that has already been done if
we start a new RC with these critical fixes included.

What do you think?

Thanks,
Gordon

On Thu, Feb 9, 2023 at 3:26 PM Tzu-Li (Gordon) Tai 
wrote:

> +1 (binding)
>
> - Verified legals (license headers and root LICENSE / NOTICE file). AFAICT
> no dependencies require explicit acknowledgement in the NOTICE files.
> - No binaries in staging area
> - Built source with tests
> - Verified signatures and hashes
> - Web PR changes LGTM
>
> Thanks Martijn!
>
> Cheers,
> Gordon
>
> On Mon, Feb 6, 2023 at 6:12 PM Mason Chen  wrote:
>
>> That makes sense, thanks for the clarification!
>>
>> Best,
>> Mason
>>
>> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser 
>> wrote:
>>
>> > Hi Mason,
>> >
>> > Thanks, [4] is indeed a copy-paste error and you've made the right
>> > assumption that
>> >
>> >
>> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
>> > is the correct maven central link.
>> >
>> > I think we should use FLINK-30052 to move the Kafka connector code from
>> the
>> > 1.17 release also over the Kafka connector repo (especially since
>> there's
>> > now a v3.0 branch for the Kafka connector, so it can be merged in main).
>> > When those commits have been merged, we can make a next Kafka connector
>> > release (which is equivalent to the 1.17 release, which can only be done
>> > when 1.17 is done because of the split level watermark alignment) and
>> then
>> > FLINK-30859 can be finished.
>> >
>> > Best regards,
>> >
>> > Martijn
>> >
>> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen :
>> >
>> > > +1 (non-binding)
>> > >
>> > > * Verified hashes and signatures
>> > > * Verified no binaries
>> > > * Verified LICENSE and NOTICE files
>> > > * Verified poms point to 3.0.0-1.16
>> > > * Reviewed web PR
>> > > * Built from source
>> > > * Verified git tag
>> > >
>> > > I think [4] your is a copy-paste error and I did all the verification
>> > > assuming that
>> > >
>> > >
>> >
>> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
>> > > is the correct maven central link.
>> > >
>> > > Regarding the release notes, should we close
>> > > https://issues.apache.org/jira/browse/FLINK-30052 and link it there?
>> > I've
>> > > created https://issues.apache.org/jira/browse/FLINK-30859 to remove
>> the
>> > > existing code from the master branch.
>> > >
>> > > Best,
>> > > Mason
>> > >
>> > > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser <
>> martijnvis...@apache.org
>> > >
>> > > wrote:
>> > >
>> > > > Hi everyone,
>> > > > Please review and vote on the release candidate #1 for
>> > > > flink-connector-kafka version 3.0.0, as follows:
>> > > > [ ] +1, Approve the release
>> > > > [ ] -1, Do not approve the release (please provide specific
>> comments)
>> > > >
>> > > > Note: this is the same code as the Kafka connector for the Flink
>> 1.16
>> > > > release.
>> > > >
>> > > > 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
>> > > > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
>> > > > * all artifacts to be deployed to the Maven Central Repository [4],
>> > > > * source code tag v3.0.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,
>> > > > Release Manager
>> > > >
>> > > > [1]
>> > > >
>> > > >
>> > >
>> >
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
>> > > > [2]
>> > > >
>> > > >
>> > >
>> >
>> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
>> > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
>> > > > [4]
>> > > >
>> > > >
>> > >
>> >
>> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
>> > > > [5]
>> > > >
>> > https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
>> > > > [6] https://github.com/apache/flink-web/pull/606
>> > > >
>> > >
>> >
>>
>


Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-02-09 Thread Tzu-Li (Gordon) Tai
+1 (binding)

- Verified legals (license headers and root LICENSE / NOTICE file). AFAICT
no dependencies require explicit acknowledgement in the NOTICE files.
- No binaries in staging area
- Built source with tests
- Verified signatures and hashes
- Web PR changes LGTM

Thanks Martijn!

Cheers,
Gordon

On Mon, Feb 6, 2023 at 6:12 PM Mason Chen  wrote:

> That makes sense, thanks for the clarification!
>
> Best,
> Mason
>
> On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser 
> wrote:
>
> > Hi Mason,
> >
> > Thanks, [4] is indeed a copy-paste error and you've made the right
> > assumption that
> >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > is the correct maven central link.
> >
> > I think we should use FLINK-30052 to move the Kafka connector code from
> the
> > 1.17 release also over the Kafka connector repo (especially since there's
> > now a v3.0 branch for the Kafka connector, so it can be merged in main).
> > When those commits have been merged, we can make a next Kafka connector
> > release (which is equivalent to the 1.17 release, which can only be done
> > when 1.17 is done because of the split level watermark alignment) and
> then
> > FLINK-30859 can be finished.
> >
> > Best regards,
> >
> > Martijn
> >
> > Op wo 1 feb. 2023 om 09:16 schreef Mason Chen :
> >
> > > +1 (non-binding)
> > >
> > > * Verified hashes and signatures
> > > * Verified no binaries
> > > * Verified LICENSE and NOTICE files
> > > * Verified poms point to 3.0.0-1.16
> > > * Reviewed web PR
> > > * Built from source
> > > * Verified git tag
> > >
> > > I think [4] your is a copy-paste error and I did all the verification
> > > assuming that
> > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > > is the correct maven central link.
> > >
> > > Regarding the release notes, should we close
> > > https://issues.apache.org/jira/browse/FLINK-30052 and link it there?
> > I've
> > > created https://issues.apache.org/jira/browse/FLINK-30859 to remove
> the
> > > existing code from the master branch.
> > >
> > > Best,
> > > Mason
> > >
> > > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser <
> martijnvis...@apache.org
> > >
> > > wrote:
> > >
> > > > Hi everyone,
> > > > Please review and vote on the release candidate #1 for
> > > > flink-connector-kafka version 3.0.0, as follows:
> > > > [ ] +1, Approve the release
> > > > [ ] -1, Do not approve the release (please provide specific comments)
> > > >
> > > > Note: this is the same code as the Kafka connector for the Flink 1.16
> > > > release.
> > > >
> > > > 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
> > > > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> > > > * all artifacts to be deployed to the Maven Central Repository [4],
> > > > * source code tag v3.0.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,
> > > > Release Manager
> > > >
> > > > [1]
> > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
> > > > [2]
> > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
> > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > [4]
> > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
> > > > [5]
> > > >
> > https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
> > > > [6] https://github.com/apache/flink-web/pull/606
> > > >
> > >
> >
>


Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-02-06 Thread Mason Chen
That makes sense, thanks for the clarification!

Best,
Mason

On Wed, Feb 1, 2023 at 7:16 AM Martijn Visser 
wrote:

> Hi Mason,
>
> Thanks, [4] is indeed a copy-paste error and you've made the right
> assumption that
>
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> is the correct maven central link.
>
> I think we should use FLINK-30052 to move the Kafka connector code from the
> 1.17 release also over the Kafka connector repo (especially since there's
> now a v3.0 branch for the Kafka connector, so it can be merged in main).
> When those commits have been merged, we can make a next Kafka connector
> release (which is equivalent to the 1.17 release, which can only be done
> when 1.17 is done because of the split level watermark alignment) and then
> FLINK-30859 can be finished.
>
> Best regards,
>
> Martijn
>
> Op wo 1 feb. 2023 om 09:16 schreef Mason Chen :
>
> > +1 (non-binding)
> >
> > * Verified hashes and signatures
> > * Verified no binaries
> > * Verified LICENSE and NOTICE files
> > * Verified poms point to 3.0.0-1.16
> > * Reviewed web PR
> > * Built from source
> > * Verified git tag
> >
> > I think [4] your is a copy-paste error and I did all the verification
> > assuming that
> >
> >
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> > is the correct maven central link.
> >
> > Regarding the release notes, should we close
> > https://issues.apache.org/jira/browse/FLINK-30052 and link it there?
> I've
> > created https://issues.apache.org/jira/browse/FLINK-30859 to remove the
> > existing code from the master branch.
> >
> > Best,
> > Mason
> >
> > On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser  >
> > wrote:
> >
> > > Hi everyone,
> > > Please review and vote on the release candidate #1 for
> > > flink-connector-kafka version 3.0.0, as follows:
> > > [ ] +1, Approve the release
> > > [ ] -1, Do not approve the release (please provide specific comments)
> > >
> > > Note: this is the same code as the Kafka connector for the Flink 1.16
> > > release.
> > >
> > > 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
> > > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> > > * all artifacts to be deployed to the Maven Central Repository [4],
> > > * source code tag v3.0.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,
> > > Release Manager
> > >
> > > [1]
> > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
> > > [2]
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
> > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > [4]
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
> > > [5]
> > >
> https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
> > > [6] https://github.com/apache/flink-web/pull/606
> > >
> >
>


Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-02-01 Thread Martijn Visser
Hi Mason,

Thanks, [4] is indeed a copy-paste error and you've made the right
assumption that
https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
is the correct maven central link.

I think we should use FLINK-30052 to move the Kafka connector code from the
1.17 release also over the Kafka connector repo (especially since there's
now a v3.0 branch for the Kafka connector, so it can be merged in main).
When those commits have been merged, we can make a next Kafka connector
release (which is equivalent to the 1.17 release, which can only be done
when 1.17 is done because of the split level watermark alignment) and then
FLINK-30859 can be finished.

Best regards,

Martijn

Op wo 1 feb. 2023 om 09:16 schreef Mason Chen :

> +1 (non-binding)
>
> * Verified hashes and signatures
> * Verified no binaries
> * Verified LICENSE and NOTICE files
> * Verified poms point to 3.0.0-1.16
> * Reviewed web PR
> * Built from source
> * Verified git tag
>
> I think [4] your is a copy-paste error and I did all the verification
> assuming that
>
> https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
> is the correct maven central link.
>
> Regarding the release notes, should we close
> https://issues.apache.org/jira/browse/FLINK-30052 and link it there? I've
> created https://issues.apache.org/jira/browse/FLINK-30859 to remove the
> existing code from the master branch.
>
> Best,
> Mason
>
> On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser 
> wrote:
>
> > Hi everyone,
> > Please review and vote on the release candidate #1 for
> > flink-connector-kafka version 3.0.0, as follows:
> > [ ] +1, Approve the release
> > [ ] -1, Do not approve the release (please provide specific comments)
> >
> > Note: this is the same code as the Kafka connector for the Flink 1.16
> > release.
> >
> > 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
> > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> > * all artifacts to be deployed to the Maven Central Repository [4],
> > * source code tag v3.0.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,
> > Release Manager
> >
> > [1]
> >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
> > [2]
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
> > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > [4]
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
> > [5]
> > https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
> > [6] https://github.com/apache/flink-web/pull/606
> >
>


Re: [VOTE] Release flink-connector-kafka, release candidate #1

2023-02-01 Thread Mason Chen
+1 (non-binding)

* Verified hashes and signatures
* Verified no binaries
* Verified LICENSE and NOTICE files
* Verified poms point to 3.0.0-1.16
* Reviewed web PR
* Built from source
* Verified git tag

I think [4] your is a copy-paste error and I did all the verification
assuming that
https://repository.apache.org/content/repositories/orgapacheflink-1582/org/apache/flink/
is the correct maven central link.

Regarding the release notes, should we close
https://issues.apache.org/jira/browse/FLINK-30052 and link it there? I've
created https://issues.apache.org/jira/browse/FLINK-30859 to remove the
existing code from the master branch.

Best,
Mason

On Tue, Jan 31, 2023 at 6:23 AM Martijn Visser 
wrote:

> Hi everyone,
> Please review and vote on the release candidate #1 for
> flink-connector-kafka version 3.0.0, as follows:
> [ ] +1, Approve the release
> [ ] -1, Do not approve the release (please provide specific comments)
>
> Note: this is the same code as the Kafka connector for the Flink 1.16
> release.
>
> 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
> A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> * all artifacts to be deployed to the Maven Central Repository [4],
> * source code tag v3.0.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,
> Release Manager
>
> [1]
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
> [2]
>
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> [4]
>
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
> [5]
> https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
> [6] https://github.com/apache/flink-web/pull/606
>


[VOTE] Release flink-connector-kafka, release candidate #1

2023-01-31 Thread Martijn Visser
Hi everyone,
Please review and vote on the release candidate #1 for
flink-connector-kafka version 3.0.0, as follows:
[ ] +1, Approve the release
[ ] -1, Do not approve the release (please provide specific comments)

Note: this is the same code as the Kafka connector for the Flink 1.16
release.

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
A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
* all artifacts to be deployed to the Maven Central Repository [4],
* source code tag v3.0.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,
Release Manager

[1]
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352577
[2]
https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1
[3] https://dist.apache.org/repos/dist/release/flink/KEYS
[4]
https://dist.apache.org/repos/dist/dev/flink/flink-connector-kafka-3.0.0-rc1/
[5] https://github.com/apache/flink-connector-kafka/releases/tag/v3.0.0-rc1
[6] https://github.com/apache/flink-web/pull/606