Here are the links:

https://github.com/apache/kafka/commit/8a417c89d2f0b7861b2dec26f02e4e302b64b604
(trunk)
https://github.com/apache/kafka/commit/c47c3b0b583a849fdf3ed0a06835427a2801950a
(0.10.0)

Ismael

On Wed, Sep 14, 2016 at 4:38 PM, feifei hsu <easyf...@gmail.com> wrote:

> Hi, Ismael
>    So many thanks for the quick reply
>    I checked the trunk tree at github, I did not see the merge.
>    did I make some mistake? sorry for that.
>
> for example, the pull 1598. file related to the LogSegment.scala. the one
> of the PR add try catch to close the leaking resource. but I did not see
> the code is at trunk. :-(
>  https://github.com/heroku/kafka/blob/trunk/core/src/main/scala/kafka/log/
> LogSegment.scala#L189
> https://github.com/apache/kafka/pull/1598/files#r70071062
>
>
> On Wed, Sep 14, 2016 at 5:06 AM, Ismael Juma <mli...@juma.me.uk> wrote:
>
>> Hi,
>>
>> We did merge the PR to trunk and 0.10.0.
>>
>> Ismael
>>
>> On Wed, Sep 14, 2016 at 9:21 AM, feifei hsu <easyf...@gmail.com> wrote:
>>
>>> Hi Tom and Ismael.
>>>    I am following the  kafka-3933. the memory leak. but I did not see
>>> the pr #1598 #1614 #1660 are merged into the trunk.
>>> Do you know what the current status?
>>>   So many thanks.
>>>    We are also thinking backport it to 0.9.0.1
>>>
>>> --easy
>>>
>>
>>
>

Reply via email to