Build failed in Jenkins: KafkaPreCommit #146

2015-07-13 Thread Apache Jenkins Server
See https://builds.apache.org/job/KafkaPreCommit/146/changes Changes: [junrao] kafka-972; MetadataRequest returns stale list of brokers; patched by Ashish Singh; reviewed by Jun Rao -- Started by an SCM change Building remotely on ubuntu3 (Ubuntu ubuntu

Re: Review Request 36333: Patch for KAFKA-2123

2015-07-13 Thread Jason Gustafson
On July 14, 2015, 12:32 a.m., Ewen Cheslack-Postava wrote: clients/src/main/java/org/apache/kafka/clients/consumer/KafkaConsumer.java, line 968 https://reviews.apache.org/r/36333/diff/1-2/?file=1002921#file1002921line968 Hmm, this seems like very different behavior from before.

[jira] [Updated] (KAFKA-2123) Make new consumer offset commit API use callback + future

2015-07-13 Thread Jason Gustafson (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson updated KAFKA-2123: --- Attachment: KAFKA-2123_2015-07-13_18:45:08.patch Make new consumer offset commit API use

Re: Review Request 36333: Patch for KAFKA-2123

2015-07-13 Thread Jason Gustafson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36333/ --- (Updated July 14, 2015, 1:45 a.m.) Review request for kafka. Bugs:

Re: [Discussion] Limitations on topic names

2015-07-13 Thread Magnus Edenhill
Hi, since dots seem to be a problem on the metrics side, why not let the metrics side handle it by escaping troublesome characters? E.g. foo.my\.topic.feh Let's not push the problem upstream. Replacing . with another set of allowed characters __ seems like a bad idea since it is ambigious:

[jira] [Commented] (KAFKA-2275) Add a ListTopic() API to the new consumer

2015-07-13 Thread Guozhang Wang (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14624307#comment-14624307 ] Guozhang Wang commented on KAFKA-2275: -- [~mtomcat_sslavic] partitionsFor() can only

<    1   2