Re: Review Request 33049: Patch for KAFKA-2084

2015-08-13 Thread Aditya Auradkar
/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-13 Thread Aditya Auradkar
--- On Aug. 14, 2015, 2:09 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-13 Thread Aditya Auradkar
/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-13 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049/ --- (Updated Aug. 11, 2015, 4:58 a.m

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-12 Thread Aditya Auradkar
://reviews.apache.org/r/33049/#review95035 --- On Aug. 11, 2015, 4:58 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Kafka Indentation

2015-08-11 Thread Aditya Auradkar
Bump. Anyone else have an opinion? Neha/Jay - You've made your thoughts clear. Any thoughts on how/if we make any changes? Thanks, Aditya On Fri, Jul 24, 2015 at 10:32 AM, Aditya Auradkar aaurad...@linkedin.com wrote: I'm with Neha on this one. I don't have a strong preference on 2 vs 4

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33378: Patch for KAFKA-2136

2015-08-10 Thread Aditya Auradkar
at once after KAFKA-2084 is committed because I will need to rebase after that. - Aditya Auradkar On July 13, 2015, 8:36 p.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https

[jira] [Commented] (KAFKA-2084) byte rate metrics per client ID (producer and consumer)

2015-08-10 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14680409#comment-14680409 ] Aditya Auradkar commented on KAFKA-2084: [~junrao][~jjkoshy] I think I've

Re: Review Request 33378: Patch for KAFKA-2136

2015-08-10 Thread Aditya Auradkar
, callback) { // add to delay queue // pass in the computed throttle time to the callback. } If I do that, I dont need to pass the throttleTime to the responseCallback in ReplicaManager - Aditya Auradkar On July 13, 2015, 8:36 p.m., Aditya Auradkar wrote

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

[jira] [Created] (KAFKA-2419) Allow certain Sensors to be garbage collected after inactivity

2015-08-10 Thread Aditya Auradkar (JIRA)
Aditya Auradkar created KAFKA-2419: -- Summary: Allow certain Sensors to be garbage collected after inactivity Key: KAFKA-2419 URL: https://issues.apache.org/jira/browse/KAFKA-2419 Project: Kafka

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
--- On Aug. 10, 2015, 8:49 p.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049

[jira] [Updated] (KAFKA-2419) Allow certain Sensors to be garbage collected after inactivity

2015-08-10 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2419?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aditya Auradkar updated KAFKA-2419: --- Description: Currently, metrics cannot be removed once registered. Implement a feature

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
. Finally, it seems that delayTime only makes sense for rates and not for anything else. Perhaps we can at least add a comment. Aditya Auradkar wrote: Hey Jun - Can you elaborate a little? How would we use the current time exactly? It is not clear to me how subtracting

[jira] [Created] (KAFKA-2420) Merge the Throttle time computation for Quotas and Throttler

2015-08-10 Thread Aditya Auradkar (JIRA)
Aditya Auradkar created KAFKA-2420: -- Summary: Merge the Throttle time computation for Quotas and Throttler Key: KAFKA-2420 URL: https://issues.apache.org/jira/browse/KAFKA-2420 Project: Kafka

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-10 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-07 Thread Aditya Auradkar
/kafka/server/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-07 Thread Aditya Auradkar
-mail. To reply, visit: https://reviews.apache.org/r/33049/#review94333 --- On Aug. 5, 2015, 2:08 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-07 Thread Aditya Auradkar
. - Aditya --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049/#review94412 --- On Aug. 5, 2015, 2:08 a.m., Aditya Auradkar wrote

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-07 Thread Aditya Auradkar
Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

[jira] [Commented] (KAFKA-1229) Reload broker config without a restart

2015-08-05 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-1229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14659071#comment-14659071 ] Aditya Auradkar commented on KAFKA-1229: Hey [~vamsi360], We recently did

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-04 Thread Aditya Auradkar
Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-04 Thread Aditya Auradkar
/ThrottledResponseExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-04 Thread Aditya Auradkar
Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-08-04 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654505#comment-14654505 ] Aditya Auradkar commented on KAFKA-2205: Thanks Jun. 1. Filed https

[jira] [Created] (KAFKA-2404) Delete config znode when config values are empty

2015-08-04 Thread Aditya Auradkar (JIRA)
Aditya Auradkar created KAFKA-2404: -- Summary: Delete config znode when config values are empty Key: KAFKA-2404 URL: https://issues.apache.org/jira/browse/KAFKA-2404 Project: Kafka Issue

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-04 Thread Aditya Auradkar
--- On June 30, 2015, 12:54 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049

Re: Review Request 33049: Patch for KAFKA-2084

2015-08-04 Thread Aditya Auradkar
/#review93965 --- On June 30, 2015, 12:54 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049

Re: [DISCUSS] KIP-28 - Add a transform client for data processing

2015-07-30 Thread Aditya Auradkar
Personally, I prefer KafkaStreams just because it sounds nicer. For the reasons identified above, KafkaProcessor or KProcessor is more apt but sounds less catchy (IMO). I also think we should prefix with Kafka (rather than K) because we will then have 3 clients: KafkaProducer, KafkaConsumer and

[jira] [Assigned] (KAFKA-2255) Missing documentation for max.in.flight.requests.per.connection

2015-07-28 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aditya Auradkar reassigned KAFKA-2255: -- Assignee: Aditya Auradkar Missing documentation

[jira] [Commented] (KAFKA-2255) Missing documentation for max.in.flight.requests.per.connection

2015-07-28 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14645166#comment-14645166 ] Aditya Auradkar commented on KAFKA-2255: This was brought up during a mailing list

Re: Kafka Consumer thoughts

2015-07-28 Thread Aditya Auradkar
Great discussion everyone! One general comment on the sync/async API's on the new consumer. I think the producer tackles sync vs async API's well. For API's that can either be sync or async, can we simply return a future? That seems more elegant for the API's that make sense either in both

Re: [DISCUSS] KIP-28 - Add a transform client for data processing

2015-07-27 Thread Aditya Auradkar
+1 on comparison with existing solutions. On a high level, it seems nice to have a transform library inside Kafka.. a lot of the building blocks are already there to build a stream processing framework. However the details are tricky to get right I think this discussion will get a lot more

Re: Review Request 36871: Patch for KAFKA-2381

2015-07-27 Thread Aditya Auradkar
233) https://reviews.apache.org/r/36871/#comment147535 consider closing this in a finally. A failing test can cause incorrect tear down of the test - Aditya Auradkar On July 28, 2015, 12:56 a.m., Ashish Singh wrote

Re: Kafka Indentation

2015-07-24 Thread Aditya Auradkar
I'm with Neha on this one. I don't have a strong preference on 2 vs 4 but I do think that consistency is more important. It makes writing code a bit easier especially since patches are increasingly likely to touch both Java and Scala code and it's nice to not think about formatting certain files

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-24 Thread Aditya Auradkar
be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-24 Thread Aditya Auradkar
be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-07-24 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14641313#comment-14641313 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] One more time :). Addressed

Re: Query about LogConfigs

2015-07-24 Thread Aditya Auradkar
Is there actually a use case where we need log.retention.ms? In most cases, people would want to retain their logs for at least a few minutes I'd think. Aditya On Fri, Jul 24, 2015 at 6:49 PM, Gwen Shapira gshap...@cloudera.com wrote: Backward compatibility, I think. At least the ms one is

Kafka Indentation

2015-07-23 Thread Aditya Auradkar
Hey everyone, I was wondering if it is possible to standardize indentation across clients and core. As an example, all the java code uses 4 spaces and the scala code 2. As we increasingly share code between clients and core, I think consistency would be super useful. Thoughts? Aditya

Re: Review Request 36722: Patch for KAFKA-2332

2015-07-23 Thread Aditya Auradkar
are adding in 2136 for the new producer and consumer? - Aditya Auradkar On July 23, 2015, 7:12 a.m., Dong Lin wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36722

Re: Round Robin Among Consumers

2015-07-22 Thread Aditya Auradkar
Hi, Why not simply have as many partitions as the set of consumers you want to round robin across? Aditya On Wed, Jul 22, 2015 at 2:37 PM, Ashish Singh asi...@cloudera.com wrote: Hey, don't you think that would be against the basic ordering guarantees Kafka provides? On Wed, Jul 22, 2015

Re: Submitting a patch (Jira errors)

2015-07-21 Thread Aditya Auradkar
Did you setup your jira.ini? On Tue, Jul 21, 2015 at 11:52 AM, Mayuresh Gharat gharatmayures...@gmail.com wrote: Hi, I had to clean up existing kafka repo on my linux box and start with a fresh one. I followed the instructions here :

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-07-17 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14631772#comment-14631772 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] Another patch ready

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-17 Thread Aditya Auradkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34554/#review92008 --- On July 14, 2015, 5:37 p.m., Aditya Auradkar wrote

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-17 Thread Aditya Auradkar
be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-17 Thread Aditya Auradkar
a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-17 Thread Aditya Auradkar
by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-17 Thread Aditya Auradkar
be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-07-14 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14626814#comment-14626814 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] Thanks! I addressed your

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-14 Thread Aditya Auradkar
: https://reviews.apache.org/r/34554/#review91038 --- On July 8, 2015, 2:14 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-14 Thread Aditya Auradkar
8a871cfaf6a534acd1def06a5ac95b5c985b024c Diff: https://reviews.apache.org/r/34554/diff/ Testing --- 1. Added new testcases for new code. 2. Verified that both topic and client configs can be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-14 Thread Aditya Auradkar
cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-14 Thread Aditya Auradkar
Gustafson kafka-2248; Use Apache Rat to enforce copyright headers; patched by Ewen Cheslack-Postava; reviewed by Gwen Shapira, Joel Joshy and Jun Rao kafka-2132; Move Log4J appender to a separate module; patched by Ashish Singh; reviewed by Gwen Shapira, Aditya Auradkar and Jun Rao KAFKA-2314

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-14 Thread Aditya Auradkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34554/#review91038 --- On July 14, 2015, 5:37 p.m., Aditya Auradkar wrote: --- This is an automatically

Re: Review Request 33378: Patch for KAFKA-2136

2015-07-13 Thread Aditya Auradkar
/#review91319 --- On July 1, 2015, 2:44 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33378

Re: Review Request 33378: Patch for KAFKA-2136

2015-07-13 Thread Aditya Auradkar
, Aditya Auradkar

Re: Review Request 33378: Patch for KAFKA-2136

2015-07-13 Thread Aditya Auradkar
do you think? Aditya Auradkar wrote: I think that sounds reasonable.. I initially decided against it in my patch because I thought of this as an incentive to upgrade. Any concerns if I submit a subsequent RB for this immediately after this is committed? Joel Koshy wrote: I think

[jira] [Created] (KAFKA-2332) Add quota metrics to old producer and consumer

2015-07-13 Thread Aditya Auradkar (JIRA)
Aditya Auradkar created KAFKA-2332: -- Summary: Add quota metrics to old producer and consumer Key: KAFKA-2332 URL: https://issues.apache.org/jira/browse/KAFKA-2332 Project: Kafka Issue Type

Re: Review Request 33378: Patch for KAFKA-2136

2015-07-13 Thread Aditya Auradkar
5717165f2344823fabe8f7cfafae4bb8af2d949a core/src/test/scala/unit/kafka/server/ReplicaManagerTest.scala 00d59337a99ac135e8689bd1ecd928f7b1423d79 Diff: https://reviews.apache.org/r/33378/diff/ Testing --- New tests added Thanks, Aditya Auradkar

[jira] [Updated] (KAFKA-2332) Add quota metrics to old producer and consumer

2015-07-13 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aditya Auradkar updated KAFKA-2332: --- Issue Type: Sub-task (was: Improvement) Parent: KAFKA-2083 Add quota metrics to old

Re: Review Request 33378: Patch for KAFKA-2136

2015-07-13 Thread Aditya Auradkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33378/#review91319 --- On July 13, 2015, 8:36 p.m., Aditya Auradkar wrote: --- This is an automatically

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-07 Thread Aditya Auradkar
8a871cfaf6a534acd1def06a5ac95b5c985b024c Diff: https://reviews.apache.org/r/34554/diff/ Testing --- 1. Added new testcases for new code. 2. Verified that both topic and client configs can be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-07-07 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14617849#comment-14617849 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] - Addressed your comments. Can

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-07 Thread Aditya Auradkar
/r/34554/#review90622 --- On July 2, 2015, 1:39 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34554

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-07 Thread Aditya Auradkar
Thanks, Aditya Auradkar

RE: [ANNOUNCE] New Committer

2015-07-06 Thread Aditya Auradkar
Congratulations Gwen! Aditya From: Ashish Singh [asi...@cloudera.com] Sent: Monday, July 06, 2015 6:16 PM To: dev@kafka.apache.org Subject: Re: [ANNOUNCE] New Committer Congrats Gwen! On Monday, July 6, 2015, Joe Stein joe.st...@stealth.ly wrote: I am

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-07-01 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14611311#comment-14611311 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] - Addressed your comments. Can

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-01 Thread Aditya Auradkar
/ Testing --- 1. Added new testcases for new code. 2. Verified that both topic and client configs can be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 34554: Patch for KAFKA-2205

2015-07-01 Thread Aditya Auradkar
/DynamicConfigChangeTest.scala 7877f6ca1845c2edbf96d4a9783a07a552db8f07 Diff: https://reviews.apache.org/r/34554/diff/ Testing --- 1. Added new testcases for new code. 2. Verified that both topic and client configs can be changed dynamically by starting a local cluster Thanks, Aditya Auradkar

Re: Review Request 33378: Patch for KAFKA-2136

2015-06-30 Thread Aditya Auradkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33378/#review89429 --- On July 1, 2015, 2:44 a.m., Aditya Auradkar wrote

Re: Review Request 33378: Patch for KAFKA-2136

2015-06-30 Thread Aditya Auradkar
/RequestResponseSerializationTest.scala 5717165f2344823fabe8f7cfafae4bb8af2d949a core/src/test/scala/unit/kafka/server/ReplicaManagerTest.scala 00d59337a99ac135e8689bd1ecd928f7b1423d79 Diff: https://reviews.apache.org/r/33378/diff/ Testing --- New tests added Thanks, Aditya Auradkar

Re: Review Request 33378: Patch for KAFKA-2136

2015-06-30 Thread Aditya Auradkar
/kafka/server/ReplicaManagerTest.scala 00d59337a99ac135e8689bd1ecd928f7b1423d79 Diff: https://reviews.apache.org/r/33378/diff/ Testing --- New tests added Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-06-29 Thread Aditya Auradkar
--- On June 30, 2015, 12:53 a.m., Aditya Auradkar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049

Re: Review Request 33049: Patch for KAFKA-2084

2015-06-29 Thread Aditya Auradkar
/server/ThrottledRequestExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-06-29 Thread Aditya Auradkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33049/#review88180 --- On June 30, 2015, 12:54 a.m., Aditya Auradkar wrote

RE: Help Us Nominate Apache Kafka for a 2015 Bossie (Best of OSS) Award - Due June 30th

2015-06-28 Thread Aditya Auradkar
Sent :) From: Gwen Shapira [gshap...@cloudera.com] Sent: Friday, June 26, 2015 11:53 AM To: us...@kafka.apache.org Cc: dev@kafka.apache.org Subject: Re: Help Us Nominate Apache Kafka for a 2015 Bossie (Best of OSS) Award - Due June 30th Sent! Thanks for

RE: [VOTE] KIP-23 - Add JSON/CSV output and looping options to ConsumerGroupCommand

2015-06-23 Thread Aditya Auradkar
Hey Ashish, Last hangout, I think we discussed adding a num-iterations parameter to the script. Do you plan to support that? Aditya From: Neha Narkhede [n...@confluent.io] Sent: Tuesday, June 23, 2015 11:24 AM To: dev@kafka.apache.org Cc: Jun Rao

[jira] [Commented] (KAFKA-2293) IllegalFormatConversionException in Partition.scala

2015-06-22 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596298#comment-14596298 ] Aditya Auradkar commented on KAFKA-2293: [~junrao] Can you take a look

[jira] [Commented] (KAFKA-2205) Generalize TopicConfigManager to handle multiple entity configs

2015-06-22 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596965#comment-14596965 ] Aditya Auradkar commented on KAFKA-2205: [~junrao] - Can you review this patch

[jira] [Commented] (KAFKA-2238) KafkaMetricsConfig cannot be configured in broker (KafkaConfig)

2015-06-20 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14594702#comment-14594702 ] Aditya Auradkar commented on KAFKA-2238: [~junrao] - KafkaConfig seems

[jira] [Created] (KAFKA-2287) Add metric to track the number of clientId throttled

2015-06-19 Thread Aditya Auradkar (JIRA)
Aditya Auradkar created KAFKA-2287: -- Summary: Add metric to track the number of clientId throttled Key: KAFKA-2287 URL: https://issues.apache.org/jira/browse/KAFKA-2287 Project: Kafka Issue

RE: Guozhang Wang elected to Kafka PMC

2015-06-16 Thread Aditya Auradkar
Congrats Guozhang! From: Ashish Singh [asi...@cloudera.com] Sent: Monday, June 15, 2015 10:53 PM To: dev@kafka.apache.org Cc: Jun Rao Subject: Re: Guozhang Wang elected to Kafka PMC Congrats Guozhang! On Mon, Jun 15, 2015 at 10:20 PM, Sriharsha

Re: Review Request 35437: Patch for KAFKA-2202

2015-06-14 Thread Aditya Auradkar
/35437/#comment140291 can probably do this inline case _: ConsumerEx = consumerTimeout.set(true) - Aditya Auradkar On June 14, 2015, 11:27 a.m., Manikumar Reddy O wrote: --- This is an automatically generated e-mail

Re: Review Request 33049: Patch for KAFKA-2084

2015-06-12 Thread Aditya Auradkar
/ThrottledRequestExpirationTest.scala PRE-CREATION Diff: https://reviews.apache.org/r/33049/diff/ Testing --- Thanks, Aditya Auradkar

Re: Review Request 33049: Patch for KAFKA-2084

2015-06-12 Thread Aditya Auradkar
and Jun Rao. Bugs: KAFKA-2084 https://issues.apache.org/jira/browse/KAFKA-2084 Repository: kafka Description (updated) --- Signed-off-by: Aditya Auradkar aaurad...@linkedin.com Addressing Joel's comments Minor imports changes Added testcase to verify that replication traffic

RE: [DISCUSS] KIP-4 - Command line and centralized administrative operations (Thread 2)

2015-06-12 Thread Aditya Auradkar
On Thu, Jun 11, 2015 at 06:36:36PM +, Aditya Auradkar wrote: Andrii, Do we need a new voting thread for this KIP? The last round of votes had 3 binding +1's but there's been a fair amount of discussion since then. Aditya From: Aditya Auradkar

Re: Review Request 33614: Patch for KAFKA-2132

2015-06-11 Thread Aditya Auradkar
here? - Aditya Auradkar On April 30, 2015, 10:53 p.m., Ashish Singh wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33614

RE: [DISCUSS] KIP-25 System test improvements

2015-06-11 Thread Aditya Auradkar
Hi Geoffrey, Thanks for the writeup. Couple of questions: - Is it possible to configure suites using ducktape? For example: assume all the tests in system_tests have been migrated to ducktape. Can I run a subset of all tests grouped by functional areas i.e. replication, broker failure etc. -

[jira] [Updated] (KAFKA-2136) Client side protocol changes to return quota delays

2015-06-11 Thread Aditya Auradkar (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-2136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aditya Auradkar updated KAFKA-2136: --- Labels: quotas (was: ) Client side protocol changes to return quota delays

<    1   2   3   4   5   >