[ 
https://issues.apache.org/jira/browse/KAFKA-385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13434332#comment-13434332
 ] 

Neha Narkhede commented on KAFKA-385:
-------------------------------------

Patch v2 doesn't apply cleanly on a fresh checkout of 0.8 -

nnarkhed-ld:kafka-385 nnarkhed$ patch -p0 -i 
~/Projects/kafka-patches/KAFKA-385-v2.patch 
patching file core/src/main/scala/kafka/Kafka.scala
patching file core/src/main/scala/kafka/api/FetchResponse.scala
patching file core/src/main/scala/kafka/metrics/KafkaMetrics.scala
patching file core/src/main/scala/kafka/metrics/KafkaMetricsConfigShared.scala
patching file core/src/main/scala/kafka/metrics/KafkaMetricsGroup.scala
patching file core/src/main/scala/kafka/server/KafkaApis.scala
Hunk #4 FAILED at 160.
Hunk #12 FAILED at 360.
2 out of 24 hunks FAILED -- saving rejects to file 
core/src/main/scala/kafka/server/KafkaApis.scala.rej
patching file core/src/main/scala/kafka/server/KafkaConfig.scala
Hunk #1 succeeded at 24 with fuzz 2 (offset 2 lines).
Hunk #2 succeeded at 36 with fuzz 1 (offset 2 lines).
Hunk #3 succeeded at 139 (offset 2 lines).
patching file core/src/main/scala/kafka/server/RequestPurgatory.scala
patching file core/src/main/scala/kafka/utils/Pool.scala
patching file core/src/main/scala/kafka/utils/Utils.scala
Hunk #1 succeeded at 502 (offset 1 line).
patching file core/src/test/scala/unit/kafka/integration/LogCorruptionTest.scala
patching file core/src/test/scala/unit/kafka/integration/TopicMetadataTest.scala
patching file core/src/test/scala/unit/kafka/server/RequestPurgatoryTest.scala
patching file project/build/KafkaProject.scala

                
> RequestPurgatory enhancements - expire/checkSatisfy issue; add jmx beans
> ------------------------------------------------------------------------
>
>                 Key: KAFKA-385
>                 URL: https://issues.apache.org/jira/browse/KAFKA-385
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Joel Koshy
>            Assignee: Joel Koshy
>             Fix For: 0.8
>
>         Attachments: example_dashboard.jpg, graphite_explorer.jpg, 
> KAFKA-385-v1.patch, KAFKA-385-v2.patch
>
>
> As discussed in KAFKA-353:
> 1 - There is potential for a client-side race condition in the 
> implementations of expire and checkSatisfied. We can just synchronize on the 
> DelayedItem.
> 2 - Would be good to add jmx beans to facilitate monitoring RequestPurgatory 
> stats.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to