[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2018-07-26 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16558593#comment-16558593 ] Ismael Juma commented on KAFKA-5621: KAFKA-5886, which is another way of fixing the underlying

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-09-11 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16162199#comment-16162199 ] Ismael Juma commented on KAFKA-5621: [~sutambe], since KIP-91 has enough votes, we should probably

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-08-03 Thread Sumant Tambe (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16113590#comment-16113590 ] Sumant Tambe commented on KAFKA-5621: - Thread closed. Please share your thoughts on KIP-91 [DISCUSS]

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-08-03 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16113406#comment-16113406 ] Ismael Juma commented on KAFKA-5621: Yes, let's start the KIP discussion in the mailing list so that

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-08-02 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112129#comment-16112129 ] Apurva Mehta commented on KAFKA-5621: - [~sutambe] : one clarification. my proposal for

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108156#comment-16108156 ] Jiangjie Qin commented on KAFKA-5621: - [~ijuma] Like [~apurva] said, for users who care about orders,

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108155#comment-16108155 ] Ismael Juma commented on KAFKA-5621: [~apurva], if the producer is blocked for more than max.block.ms

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108141#comment-16108141 ] Apurva Mehta commented on KAFKA-5621: - [~ijuma], can't speak for [~becket_qin], but it seems that the

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108130#comment-16108130 ] Ismael Juma commented on KAFKA-5621: [~becket_qin], you said a couple of times that if delivery can't

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Jason Gustafson (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108095#comment-16108095 ] Jason Gustafson commented on KAFKA-5621: Interesting discussion. Apurva's suggestion on a

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108067#comment-16108067 ] Apurva Mehta commented on KAFKA-5621: - [~becket_qin] shall we then post kip-91 to the community list

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108050#comment-16108050 ] Jiangjie Qin commented on KAFKA-5621: - [~apurva] Thanks for the explanation. I think we are on the

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-31 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107971#comment-16107971 ] Apurva Mehta commented on KAFKA-5621: - Thanks for your comment Becket, and sorry for the delay in

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-27 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16103967#comment-16103967 ] Jiangjie Qin commented on KAFKA-5621: - [~apurva] I am trying to understand the following statement

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-27 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16103516#comment-16103516 ] Apurva Mehta commented on KAFKA-5621: - [~sutambe] : We could name the second config

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-27 Thread Sumant Tambe (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16103460#comment-16103460 ] Sumant Tambe commented on KAFKA-5621: - {{queue.time.ms}} sound a lot like {{linger.ms}} to me. It may

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-26 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16102176#comment-16102176 ] Jiangjie Qin commented on KAFKA-5621: - [~apurva] Yes, I agree that expiry ms is a new concept as it is

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-26 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16101932#comment-16101932 ] Jiangjie Qin commented on KAFKA-5621: - [~ijuma] [~apurva] The expiration for messages in the

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-25 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16101000#comment-16101000 ] Ismael Juma commented on KAFKA-5621: Btw, we can avoid the double retry issue by simply assuming that

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-25 Thread Apurva Mehta (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100946#comment-16100946 ] Apurva Mehta commented on KAFKA-5621: - I think there are a few points here: 1. What does

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-25 Thread Sumant Tambe (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100349#comment-16100349 ] Sumant Tambe commented on KAFKA-5621: - [~ijuma], [~apurva], It looks like this proposal will change

[jira] [Commented] (KAFKA-5621) The producer should retry expired batches when retries are enabled

2017-07-25 Thread Ismael Juma (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-5621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099910#comment-16099910 ] Ismael Juma commented on KAFKA-5621: [~jjkoshy], [~becket_qin], [~sutambe], what are your thoughts on