[jira] [Commented] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649603#comment-13649603 ] Uma Maheswara Rao G commented on BOOKKEEPER-562: Nit: looks like test

[jira] [Updated] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Flavio Junqueira (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-562: Attachment: BOOKKEEPER-562.patch There were some extra spaces in the test

[jira] [Updated] (BOOKKEEPER-578) LedgerCacheImpl is reserving 1/3 of Heap size but allocates NonHeap memory

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-578: -- Fix Version/s: 4.3.0 LedgerCacheImpl is reserving 1/3 of Heap size but allocates

[jira] [Updated] (BOOKKEEPER-608) Make SyncThread a reusable component

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-608?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-608: -- Attachment: 0001-BOOKKEEPER-608-Move-syncthread-out-of-bookie.patch Patch moves the

[jira] [Commented] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649722#comment-13649722 ] Ivan Kelly commented on BOOKKEEPER-562: --- Patch looks good. We shouldn't use

[jira] [Commented] (BOOKKEEPER-257) Ability to list all ledgers

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649728#comment-13649728 ] Ivan Kelly commented on BOOKKEEPER-257: --- This will give an OutOfMemory in the

[jira] [Updated] (BOOKKEEPER-577) BookieFailureTest uses sync/wait()/notify() incorrectly

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-577?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-577: -- Attachment: 0001-BOOKKEEPER-577-BookieFailureTest-uses-sync-wait-noti.patch Actually, a

[jira] [Commented] (BOOKKEEPER-577) BookieFailureTest uses sync/wait()/notify() incorrectly

2013-05-06 Thread Flavio Junqueira (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649738#comment-13649738 ] Flavio Junqueira commented on BOOKKEEPER-577: - Sure, this is good for me,

[jira] [Updated] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Flavio Junqueira (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-562: Attachment: BOOKKEEPER-562.patch I haven't done what I said previously, I

[jira] [Commented] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649765#comment-13649765 ] Ivan Kelly commented on BOOKKEEPER-562: --- IsClosedCallback needs a ctx object,

[jira] [Updated] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Flavio Junqueira (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-562: Attachment: BOOKKEEPER-562.patch I have fixed it according to the comments.

[jira] [Commented] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13649799#comment-13649799 ] Ivan Kelly commented on BOOKKEEPER-562: --- Patch looks good. +1. There's no need

[jira] [Updated] (BOOKKEEPER-257) Ability to list all ledgers

2013-05-06 Thread Flavio Junqueira (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Junqueira updated BOOKKEEPER-257: Attachment: BOOKKEEPER-257.patch What about this one?

[jira] [Commented] (BOOKKEEPER-562) Ability to tell if a ledger is closed or not

2013-05-06 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13650491#comment-13650491 ] Hudson commented on BOOKKEEPER-562: --- Integrated in bookkeeper-trunk #195 (See

[jira] [Commented] (BOOKKEEPER-564) Better checkpoint mechanism

2013-05-06 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13650490#comment-13650490 ] Hudson commented on BOOKKEEPER-564: --- Integrated in bookkeeper-trunk #195 (See