[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13547816#comment-13547816 ] Ivan Kelly commented on BOOKKEEPER-538: --- awaitTermination doesn't force anything

[jira] [Commented] (BOOKKEEPER-249) Revisit garbage collection algorithm in Bookie server

[ https://issues.apache.org/jira/browse/BOOKKEEPER-249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13547825#comment-13547825 ] Flavio Junqueira commented on BOOKKEEPER-249: - If this is still for 4.3.0,

[jira] [Updated] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-538: -- Attachment: 0001-BOOKKEEPER-538-Race-condition-in-BookKeeper-close.patch New patch adds tes

[jira] [Updated] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-533: -- Attachment: 0001-BOOKKEEPER-533-TestSubAfterCloseSub-fails-strangely-.patch > TestSubAf

[jira] [Updated] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-538: -- Attachment: 0001-BOOKKEEPER-538-Race-condition-in-BookKeeper-close.patch Last patch was the

[jira] [Updated] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-533: -- Attachment: (was: 0001-BOOKKEEPER-530-data-might-be-lost-during-compaction.patch)

[jira] [Comment Edited] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548356#comment-13548356 ] Ivan Kelly edited comment on BOOKKEEPER-538 at 1/9/13 10:31 AM:

[jira] [Commented] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548358#comment-13548358 ] Ivan Kelly commented on BOOKKEEPER-533: --- Ah, I moved it so that it could access

[jira] [Comment Edited] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548358#comment-13548358 ] Ivan Kelly edited comment on BOOKKEEPER-533 at 1/9/13 10:36 AM:

[jira] [Commented] (BOOKKEEPER-530) data might be lost during compaction.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548364#comment-13548364 ] Ivan Kelly commented on BOOKKEEPER-530: --- with one by one it could take a very lo

[jira] [Commented] (BOOKKEEPER-531) Cache thread should wait until old entries are collected

[ https://issues.apache.org/jira/browse/BOOKKEEPER-531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548389#comment-13548389 ] Ivan Kelly commented on BOOKKEEPER-531: --- New patch lgtm +1. Committing.

[jira] [Commented] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548390#comment-13548390 ] Hadoop QA commented on BOOKKEEPER-533: -- Testing JIRA BOOKKEEPER-533 Patch [000

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548409#comment-13548409 ] Hadoop QA commented on BOOKKEEPER-538: -- Testing JIRA BOOKKEEPER-538 Patch [000

[jira] [Commented] (BOOKKEEPER-531) Cache thread should wait until old entries are collected

[ https://issues.apache.org/jira/browse/BOOKKEEPER-531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548435#comment-13548435 ] Hudson commented on BOOKKEEPER-531: --- Integrated in bookkeeper-trunk2 #43 (See [http

[jira] [Created] (BOOKKEEPER-541) Add guava to notice file

Ivan Kelly created BOOKKEEPER-541: - Summary: Add guava to notice file Key: BOOKKEEPER-541 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-541 Project: Bookkeeper Issue Type: Bug

[jira] [Commented] (BOOKKEEPER-530) data might be lost during compaction.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548447#comment-13548447 ] Rakesh R commented on BOOKKEEPER-530: - Yeah, smaller chunks would be fine and woul

[jira] [Updated] (BOOKKEEPER-541) Add guava to notice file

[ https://issues.apache.org/jira/browse/BOOKKEEPER-541?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-541: -- Attachment: 0001-BOOKKEEPER-541-Add-guava-to-notice-file.patch Patch also adds cleanup for

[jira] [Commented] (BOOKKEEPER-541) Add guava to notice file

[ https://issues.apache.org/jira/browse/BOOKKEEPER-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548511#comment-13548511 ] Hadoop QA commented on BOOKKEEPER-541: -- Testing JIRA BOOKKEEPER-541 Patch [000

[jira] [Updated] (BOOKKEEPER-540) #stopServingSubscriber when channel is disconnected.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fangmin Lv updated BOOKKEEPER-540: -- Attachment: BOOKKEEPER-540.patch Attach a simple and straightforward patch to remove stub c

[jira] [Commented] (BOOKKEEPER-539) ClientNotSubscribedException & doesn't receive enough messages in TestThrottlingDelivery#testServerSideThrottle

[ https://issues.apache.org/jira/browse/BOOKKEEPER-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548526#comment-13548526 ] Uma Maheswara Rao G commented on BOOKKEEPER-539: If that is the case,

[jira] [Commented] (BOOKKEEPER-540) #stopServingSubscriber when channel is disconnected.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548538#comment-13548538 ] Hadoop QA commented on BOOKKEEPER-540: -- Testing JIRA BOOKKEEPER-540 Patch [BOO

[jira] [Updated] (BOOKKEEPER-530) data might be lost during compaction.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-530: -- Attachment: 0001-BOOKKEEPER-530-data-might-be-lost-during-compaction.patch Latest patch lim

[jira] [Commented] (BOOKKEEPER-530) data might be lost during compaction.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548565#comment-13548565 ] Hadoop QA commented on BOOKKEEPER-530: -- Testing JIRA BOOKKEEPER-530 Patch [000

[jira] [Updated] (BOOKKEEPER-541) Add guava to notice file

[ https://issues.apache.org/jira/browse/BOOKKEEPER-541?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-541: Issue Type: Improvement (was: Bug) > Add guava to notice file > --

[jira] [Commented] (BOOKKEEPER-541) Add guava to notice file

[ https://issues.apache.org/jira/browse/BOOKKEEPER-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548571#comment-13548571 ] Flavio Junqueira commented on BOOKKEEPER-541: - +1, I'm not sure what type

[jira] [Commented] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548581#comment-13548581 ] Flavio Junqueira commented on BOOKKEEPER-533: - The patch still looks wrong

[jira] [Updated] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-533: -- Attachment: 0001-BOOKKEEPER-533-TestSubAfterCloseSub-fails-strangely-.patch This one has it

[jira] [Commented] (BOOKKEEPER-529) stopServingSubscriber in delivery manager should remove stub callbacks in ReadAheadCache

[ https://issues.apache.org/jira/browse/BOOKKEEPER-529?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548616#comment-13548616 ] Ivan Kelly commented on BOOKKEEPER-529: --- Looks good to me +1. Committing.

[jira] [Commented] (BOOKKEEPER-541) Add guava to notice file

[ https://issues.apache.org/jira/browse/BOOKKEEPER-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548623#comment-13548623 ] Hudson commented on BOOKKEEPER-541: --- Integrated in bookkeeper-trunk2 #45 (See [http

[jira] [Commented] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548639#comment-13548639 ] Hadoop QA commented on BOOKKEEPER-533: -- Testing JIRA BOOKKEEPER-533 Patch [000

[jira] [Created] (BOOKKEEPER-542) Remove trailing spaces in IndexCorruptionTest

Flavio Junqueira created BOOKKEEPER-542: --- Summary: Remove trailing spaces in IndexCorruptionTest Key: BOOKKEEPER-542 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-542 Project: Bookkeeper

[jira] [Updated] (BOOKKEEPER-542) Remove trailing spaces in IndexCorruptionTest

[ https://issues.apache.org/jira/browse/BOOKKEEPER-542?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-542: Attachment: BOOKKEEPER-542.patch Trivial patch to remove the trailing spaces in

[jira] [Updated] (BOOKKEEPER-542) Remove trailing spaces in IndexCorruptionTest

[ https://issues.apache.org/jira/browse/BOOKKEEPER-542?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-542: Fix Version/s: 4.2.0 > Remove trailing spaces in IndexCorruptionTest >

[jira] [Commented] (BOOKKEEPER-533) TestSubAfterCloseSub fails strangely in tests

[ https://issues.apache.org/jira/browse/BOOKKEEPER-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548681#comment-13548681 ] Hudson commented on BOOKKEEPER-533: --- Integrated in bookkeeper-trunk2 #46 (See [http

[jira] [Commented] (BOOKKEEPER-542) Remove trailing spaces in IndexCorruptionTest

[ https://issues.apache.org/jira/browse/BOOKKEEPER-542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548686#comment-13548686 ] Hadoop QA commented on BOOKKEEPER-542: -- Testing JIRA BOOKKEEPER-542 Patch [BOO

[jira] [Updated] (BOOKKEEPER-513) TestMessageFilter fails periodically

[ https://issues.apache.org/jira/browse/BOOKKEEPER-513?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-513: -- Attachment: 0001-BOOKKEEPER-513-TestMessageFilter-fails-periodically.patch Patch rebased on

[jira] [Commented] (BOOKKEEPER-503) The test case of TestThrottlingDelivery#testServerSideThrottle failed sometimes

[ https://issues.apache.org/jira/browse/BOOKKEEPER-503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548694#comment-13548694 ] Ivan Kelly commented on BOOKKEEPER-503: --- The fact that deliverNextMessage is in

[jira] [Commented] (BOOKKEEPER-529) stopServingSubscriber in delivery manager should remove stub callbacks in ReadAheadCache

[ https://issues.apache.org/jira/browse/BOOKKEEPER-529?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548700#comment-13548700 ] Hudson commented on BOOKKEEPER-529: --- Integrated in bookkeeper-trunk2 #47 (See [http

[jira] [Commented] (BOOKKEEPER-513) TestMessageFilter fails periodically

[ https://issues.apache.org/jira/browse/BOOKKEEPER-513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548742#comment-13548742 ] Hadoop QA commented on BOOKKEEPER-513: -- Testing JIRA BOOKKEEPER-513 Patch [000

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548770#comment-13548770 ] Sijie Guo commented on BOOKKEEPER-538: -- checked the javadoc about 'awaitTerminati

[jira] [Commented] (BOOKKEEPER-249) Revisit garbage collection algorithm in Bookie server

[ https://issues.apache.org/jira/browse/BOOKKEEPER-249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548772#comment-13548772 ] Sijie Guo commented on BOOKKEEPER-249: -- sure, it was 4.3.0. just ensure that you

[jira] [Commented] (BOOKKEEPER-542) Remove trailing spaces in IndexCorruptionTest

[ https://issues.apache.org/jira/browse/BOOKKEEPER-542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548780#comment-13548780 ] Hudson commented on BOOKKEEPER-542: --- Integrated in bookkeeper-trunk2 #48 (See [http

[jira] [Commented] (BOOKKEEPER-249) Revisit garbage collection algorithm in Bookie server

[ https://issues.apache.org/jira/browse/BOOKKEEPER-249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548796#comment-13548796 ] Flavio Junqueira commented on BOOKKEEPER-249: - It still is 4.3.0, right? Y

[jira] [Commented] (BOOKKEEPER-539) ClientNotSubscribedException & doesn't receive enough messages in TestThrottlingDelivery#testServerSideThrottle

[ https://issues.apache.org/jira/browse/BOOKKEEPER-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548812#comment-13548812 ] Sijie Guo commented on BOOKKEEPER-539: -- I would prefer that [~jiannan] could veri

[jira] [Commented] (BOOKKEEPER-249) Revisit garbage collection algorithm in Bookie server

[ https://issues.apache.org/jira/browse/BOOKKEEPER-249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13548813#comment-13548813 ] Sijie Guo commented on BOOKKEEPER-249: -- sorry... [~fpj] yeah, it is...

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549094#comment-13549094 ] Ivan Kelly commented on BOOKKEEPER-538: --- {quote} if timeout occurs or current th

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549354#comment-13549354 ] Sijie Guo commented on BOOKKEEPER-538: -- OK. I am fine with it. +1 for the patch.

[jira] [Commented] (BOOKKEEPER-503) The test case of TestThrottlingDelivery#testServerSideThrottle failed sometimes

[ https://issues.apache.org/jira/browse/BOOKKEEPER-503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549357#comment-13549357 ] Sijie Guo commented on BOOKKEEPER-503: -- we could do it. if we want to do that way

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549360#comment-13549360 ] Uma Maheswara Rao G commented on BOOKKEEPER-538: Hi Ivan, Could you p

[jira] [Commented] (BOOKKEEPER-538) Race condition in BookKeeper#close

[ https://issues.apache.org/jira/browse/BOOKKEEPER-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549363#comment-13549363 ] Sijie Guo commented on BOOKKEEPER-538: -- [~umamaheswararao] If I understand your q

[jira] [Updated] (BOOKKEEPER-526) multiple threads for delivery manager

[ https://issues.apache.org/jira/browse/BOOKKEEPER-526?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-526: - Attachment: BOOKKEEPER-526.diff rebase the patch to latest trunk. > multiple

[jira] [Updated] (BOOKKEEPER-540) #stopServingSubscriber when channel is disconnected.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-540: - Attachment: BOOKKEEPER-540.diff helping resolving findbugs warning. > #stopS

[jira] [Commented] (BOOKKEEPER-526) multiple threads for delivery manager

[ https://issues.apache.org/jira/browse/BOOKKEEPER-526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549378#comment-13549378 ] Hadoop QA commented on BOOKKEEPER-526: -- Testing JIRA BOOKKEEPER-526 Patch [BOO

[jira] [Commented] (BOOKKEEPER-540) #stopServingSubscriber when channel is disconnected.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549386#comment-13549386 ] Hadoop QA commented on BOOKKEEPER-540: -- Testing JIRA BOOKKEEPER-540 Patch [BOO

[jira] [Created] (BOOKKEEPER-543) Read zk host list in a wrong way in hedwig server

Fangmin Lv created BOOKKEEPER-543: - Summary: Read zk host list in a wrong way in hedwig server Key: BOOKKEEPER-543 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-543 Project: Bookkeeper

[jira] [Updated] (BOOKKEEPER-543) Read zk host list in a wrong way in hedwig server

[ https://issues.apache.org/jira/browse/BOOKKEEPER-543?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fangmin Lv updated BOOKKEEPER-543: -- Attachment: BOOKKEEPER-543.patch > Read zk host list in a wrong way in hedwig server >