+1

(Non binding)

Thanks
Eno

> On 4 Oct 2019, at 15:09, Harsha Chintalapani <ka...@harsha.io> wrote:
> 
> +1 (binding).
> 
> Thanks,
> Harsha
> 
> 
>> On Fri, Oct 04, 2019 at 6:53 AM, Manikumar <manikumar.re...@gmail.com>
>> wrote:
>> 
>> Hi All,
>> 
>> Please vote here for the formal approval of this KIP.
>> https://github.com/apache/kafka/pull/6295#discussion_r328867657
>> 
>> Thanks,
>> 
>> 
>> On Wed, Oct 2, 2019 at 5:50 AM Ryanne Dolan <ryannedo...@gmail.com> wrote:
>> 
>> Hey y'all, resurrecting an old KIP for the benefit of KIP-382, which
>> depends on an additional parameter in SourceTask.commitRecord(). I've
>> updated KIP-416 according to consensus reached in PR-6295. Let's finish the
>> vote so we can formally approve this minor KIP, please!
>> 
>> Ryanne
>> 
>>> On Mon, Jan 21, 2019, 4:25 PM Ryanne Dolan <ryannedo...@gmail.com> wrote:
>>> 
>>> Andrew, I agree it's a better commitRecord, but with the slightly
>>> different semantics you mentioned. I suppose we could document that well
>>> enough that reusing the same name would be fine.
>>> 
>>> I'll resend the discussion email. Maybe it got lost somehow.
>>> 
>>> Ryanne
>>> 
>>> On Mon, Jan 21, 2019, 4:37 AM Andrew Schofield <andrew_schofield@live.
>> com
>>> wrote:
>>> 
>>>> Hi,
>>>> I'm not quite sure about the etiquette here but I wonder whether the KIP
>>>> could be improved. I think I missed the DISCUSS thread.
>>>> 
>>>> I think that really your recordLogged(SourceRecord, RecordMetadata)
>>>> method is actually a better version of commitRecord() and perhaps it
>> ought
>>>> to be an overload. This is similar to the situation in which the
>> Serializer
>>>> interface was enhanced when record headers were added.
>>>> 
>>>> public abstract class SourceTask implements Task {
>>>>  public void commitRecord(SourceRecord sourceRecord, RecordMetadata
>>>> recordMetadata) {
>>>>    this.commitRecord();
>>>>  }
>>>> 
>>>>  public void commitRecord() {
>>>>  }
>>>> }
>>>> 
>>>> Or something like that. I do understand that the KIP mentions that
>>>> recordLogged() is only called for records that are actually ACKed, but
>> it's
>>>> very similar in intent to commitRecord() in my view.
>>>> 
>>>> Just my 2 cents.
>>>> 
>>>> Andrew Schofield
>>>> IBM Event Streams
>>>> 
>>>> 
>>>> On 17/01/2019, 23:54, "Ryanne Dolan" <ryannedo...@gmail.com> wrote:
>>>> 
>>>>    Hey y'all, please vote for KIP-416 by replying +1 to this thread.
>>>> 
>>>>    Right now, there is no way for a SourceConnector/Task to know:
>>>> 
>>>>    - whether a record was successfully sent to Kafka, vs filtered out
>> or
>>>>    skipped.
>>>>    - the downstream offsets and metadata of sent records
>>>> 
>>>>    KIP-416 proposes adding a recordLogged() callback for this purpose.
>>>> 
>>>> 
>>>> https://nam05.safelinks.protection.outlook.com/
>> ?url=https%3A%2F%2Fcwiki.apache.
>> org%2Fconfluence%2Fdisplay%2FKAFKA%2FKIP-416%253A%2BNotify%2BSourceTask%2Bof%2BACK%2527d%2Boffsets%252C%2Bmetadata&amp;data=02%7C01%7C%7C9fa617754cce4bab7ba508d67cd7128f%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636833660500817715&amp;sdata=udEP27%2FrshuP5sWthvZmUIdt13whM5XqKMoia1wE93c%3D&amp;reserved=0
>>>> 
>>>>    Thanks!
>>>>    Ryanne
>>>> 
>>>> 
>>>> 
>> 
>> 

Reply via email to