Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-08-29 Thread Adam Bellemare
Hi Matt

I posted in the PR about some of the commenting changes. Overall I think
the whole thing looks good, so I would give it a:

+1 non-binding

Adam

On Tue, Aug 28, 2018 at 5:13 PM, Matt Farmer  wrote:

> Given that voting and discussion have stalled out it seems like this is a
> thing that folks aren't particularly interested in. I'll be moving the KIP
> status to abandoned unless I hear an objection in the next day or so. :)
>
> On Thu, May 31, 2018 at 12:39 PM Matt Farmer  wrote:
>
> > Bumping this again as it's been languishing for a few weeks. Would love
> to
> > get further feedback (or know for sure that this won't happen).
> >
> > On Mon, May 14, 2018 at 3:48 PM, Matt Farmer  wrote:
> >
> >> Bumping this thread.
> >>
> >> For anyone who needs a refresher the discussion thread is here:
> >> http://mail-archives.apache.org/mod_mbox/kafka-dev/201803.
> mbox/%3CCAM5dya9x---9M3uEf_wrJL5dw%2B6HLV4%3D5PfKKSTPE1vOHEWC_g%40mail.
> gmail.com%3E
> >>
> >> And there's a work in progress PR open here:
> >> https://github.com/apache/kafka/pull/5002
> >>
> >> Thanks!
> >>
> >> On Wed, Apr 25, 2018 at 1:04 PM, Matt Farmer  wrote:
> >>
> >>> Bump!
> >>>
> >>> We're currently at 1 non-binding +1.
> >>>
> >>> Still soliciting votes here. =)
> >>>
> >>> On Wed, Apr 18, 2018 at 3:41 PM, Ted Yu  wrote:
> >>>
>  +1
> 
>  On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:
> 
>  > Good afternoon/evening/morning all:
>  >
>  > I'd like to start voting on KIP-275: Indicate "isClosing" in the
>  > SinkTaskContext
>  >
>  https://cwiki.apache.org/confluence/pages/viewpage.
> action?pageId=75977607
>  >
>  > I'm going to start preparing the patch we've been using internally
>  for PR
>  > and get it up for review later this week.
>  >
>  > Thanks!
>  > Matt
>  >
> 
> >>>
> >>>
> >>
> >
>


Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-08-28 Thread Matt Farmer
Given that voting and discussion have stalled out it seems like this is a
thing that folks aren't particularly interested in. I'll be moving the KIP
status to abandoned unless I hear an objection in the next day or so. :)

On Thu, May 31, 2018 at 12:39 PM Matt Farmer  wrote:

> Bumping this again as it's been languishing for a few weeks. Would love to
> get further feedback (or know for sure that this won't happen).
>
> On Mon, May 14, 2018 at 3:48 PM, Matt Farmer  wrote:
>
>> Bumping this thread.
>>
>> For anyone who needs a refresher the discussion thread is here:
>> http://mail-archives.apache.org/mod_mbox/kafka-dev/201803.mbox/%3CCAM5dya9x---9M3uEf_wrJL5dw%2B6HLV4%3D5PfKKSTPE1vOHEWC_g%40mail.gmail.com%3E
>>
>> And there's a work in progress PR open here:
>> https://github.com/apache/kafka/pull/5002
>>
>> Thanks!
>>
>> On Wed, Apr 25, 2018 at 1:04 PM, Matt Farmer  wrote:
>>
>>> Bump!
>>>
>>> We're currently at 1 non-binding +1.
>>>
>>> Still soliciting votes here. =)
>>>
>>> On Wed, Apr 18, 2018 at 3:41 PM, Ted Yu  wrote:
>>>
 +1

 On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:

 > Good afternoon/evening/morning all:
 >
 > I'd like to start voting on KIP-275: Indicate "isClosing" in the
 > SinkTaskContext
 >
 https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=75977607
 >
 > I'm going to start preparing the patch we've been using internally
 for PR
 > and get it up for review later this week.
 >
 > Thanks!
 > Matt
 >

>>>
>>>
>>
>


Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-05-31 Thread Matt Farmer
Bumping this again as it's been languishing for a few weeks. Would love to
get further feedback (or know for sure that this won't happen).

On Mon, May 14, 2018 at 3:48 PM, Matt Farmer  wrote:

> Bumping this thread.
>
> For anyone who needs a refresher the discussion thread is here:
> http://mail-archives.apache.org/mod_mbox/kafka-dev/
> 201803.mbox/%3CCAM5dya9x---9M3uEf_wrJL5dw%2B6HLV4%
> 3D5PfKKSTPE1vOHEWC_g%40mail.gmail.com%3E
>
> And there's a work in progress PR open here: https://github.com/
> apache/kafka/pull/5002
>
> Thanks!
>
> On Wed, Apr 25, 2018 at 1:04 PM, Matt Farmer  wrote:
>
>> Bump!
>>
>> We're currently at 1 non-binding +1.
>>
>> Still soliciting votes here. =)
>>
>> On Wed, Apr 18, 2018 at 3:41 PM, Ted Yu  wrote:
>>
>>> +1
>>>
>>> On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:
>>>
>>> > Good afternoon/evening/morning all:
>>> >
>>> > I'd like to start voting on KIP-275: Indicate "isClosing" in the
>>> > SinkTaskContext
>>> > https://cwiki.apache.org/confluence/pages/viewpage.action?pa
>>> geId=75977607
>>> >
>>> > I'm going to start preparing the patch we've been using internally for
>>> PR
>>> > and get it up for review later this week.
>>> >
>>> > Thanks!
>>> > Matt
>>> >
>>>
>>
>>
>


Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-05-14 Thread Matt Farmer
Bumping this thread.

For anyone who needs a refresher the discussion thread is here:
http://mail-archives.apache.org/mod_mbox/kafka-dev/201803.mbox/%3CCAM5dya9x---9M3uEf_wrJL5dw%2B6HLV4%3D5PfKKSTPE1vOHEWC_g%40mail.gmail.com%3E

And there's a work in progress PR open here:
https://github.com/apache/kafka/pull/5002

Thanks!

On Wed, Apr 25, 2018 at 1:04 PM, Matt Farmer  wrote:

> Bump!
>
> We're currently at 1 non-binding +1.
>
> Still soliciting votes here. =)
>
> On Wed, Apr 18, 2018 at 3:41 PM, Ted Yu  wrote:
>
>> +1
>>
>> On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:
>>
>> > Good afternoon/evening/morning all:
>> >
>> > I'd like to start voting on KIP-275: Indicate "isClosing" in the
>> > SinkTaskContext
>> > https://cwiki.apache.org/confluence/pages/viewpage.action?
>> pageId=75977607
>> >
>> > I'm going to start preparing the patch we've been using internally for
>> PR
>> > and get it up for review later this week.
>> >
>> > Thanks!
>> > Matt
>> >
>>
>
>


Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-04-25 Thread Matt Farmer
Bump!

We're currently at 1 non-binding +1.

Still soliciting votes here. =)

On Wed, Apr 18, 2018 at 3:41 PM, Ted Yu  wrote:

> +1
>
> On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:
>
> > Good afternoon/evening/morning all:
> >
> > I'd like to start voting on KIP-275: Indicate "isClosing" in the
> > SinkTaskContext
> > https://cwiki.apache.org/confluence/pages/viewpage.
> action?pageId=75977607
> >
> > I'm going to start preparing the patch we've been using internally for PR
> > and get it up for review later this week.
> >
> > Thanks!
> > Matt
> >
>


Re: [VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-04-18 Thread Ted Yu
+1

On Wed, Apr 18, 2018 at 12:40 PM, Matt Farmer  wrote:

> Good afternoon/evening/morning all:
>
> I'd like to start voting on KIP-275: Indicate "isClosing" in the
> SinkTaskContext
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=75977607
>
> I'm going to start preparing the patch we've been using internally for PR
> and get it up for review later this week.
>
> Thanks!
> Matt
>


[VOTE] KIP-275 - Indicate "isClosing" in the SinkTaskContext

2018-04-18 Thread Matt Farmer
Good afternoon/evening/morning all:

I'd like to start voting on KIP-275: Indicate "isClosing" in the
SinkTaskContext
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=75977607

I'm going to start preparing the patch we've been using internally for PR
and get it up for review later this week.

Thanks!
Matt