[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Rakesh R (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13406914#comment-13406914 ] Rakesh R commented on BOOKKEEPER-247: - @Uma @Ivan bq.Here is one boundary case ca

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13406918#comment-13406918 ] Uma Maheswara Rao G commented on BOOKKEEPER-247: Current writer only c

Reg System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Rakesh R
Hi, In our test env, we have got the following problem. After analyzing found the problem occured due system time changes. Actually in our cluster we are running a periodic syncup scripts just to unify the SystemTime in all the machines. This is causing the problem and resulting ArrayIndexOut

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13406940#comment-13406940 ] Uma Maheswara Rao G commented on BOOKKEEPER-247: One more case here, t

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13406950#comment-13406950 ] Uma Maheswara Rao G commented on BOOKKEEPER-247: Even though we treat

[jira] [Updated] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uma Maheswara Rao G updated BOOKKEEPER-247: --- Attachment: BOOKKEEPER-247-1.patch Attached the patch with above change a

[jira] [Updated] (BOOKKEEPER-306) Change C++ client to use gtest for testing

2012-07-05 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-306?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Kelly updated BOOKKEEPER-306: -- Attachment: BOOKKEEPER-306.diff Check the readme for how to run the tests now. I didn't cha

[jira] [Commented] (BOOKKEEPER-294) Not able to start the bookkeeper before the ZK session timeout.

2012-07-05 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-294?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407092#comment-13407092 ] Hudson commented on BOOKKEEPER-294: --- Integrated in bookkeeper-trunk #592 (See [http

Re: Reg System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Ivan Kelly
> I agree it is not good to change the system time on a running > process, still I'm thinking we can use System.nanoTime(), this will > be giving the JVM startup time and will helpful to withstand this > kind of minor accidents(as this came in the stat calculation > logic). Good idea. Could you o

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407182#comment-13407182 ] Ivan Kelly commented on BOOKKEEPER-247: --- The first case (single entry in ledger)

RE: Reg System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Uma Maheswara Rao G
Yep, Agreed. Better to go with System.nanoTime(). Atleast we will avoid the problems in the machines where it supports monotonicness. Regards, Uma From: Ivan Kelly [iv...@apache.org] Sent: Thursday, July 05, 2012 8:14 PM To: bookkeeper-dev@zookeeper.apach

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407223#comment-13407223 ] Uma Maheswara Rao G commented on BOOKKEEPER-247: Hi Ivan, {quote} Btw,

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Ivan Kelly (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407287#comment-13407287 ] Ivan Kelly commented on BOOKKEEPER-247: --- Fixed the empty ledger case. https://g

[jira] [Commented] (BOOKKEEPER-247) Detection of under replication

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407395#comment-13407395 ] Uma Maheswara Rao G commented on BOOKKEEPER-247: Yep, you are right It

[jira] [Updated] (BOOKKEEPER-327) System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uma Maheswara Rao G updated BOOKKEEPER-327: --- Attachment: BOOKKEEPER-327.patch > System.currentTimeMillis usage in

RE: Reg System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Mridul Muralidharan
There is no guarantee that it needs to be monotonic - particularly in MP systems - both numa and smp systems : unless you tie each process to a particular core via affinity. nanotime() attempts to do a bestcase effort based on ticks (usually), while currentTimeInMillis is based on clock. Both c

[jira] [Commented] (BOOKKEEPER-327) System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407451#comment-13407451 ] Uma Maheswara Rao G commented on BOOKKEEPER-327: @Rakesh, can you plea

[jira] [Commented] (BOOKKEEPER-311) Changes in hedwig client api to support JMS spec

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407605#comment-13407605 ] Mridul Muralidharan commented on BOOKKEEPER-311: Splitting into a) c

[jira] [Updated] (BOOKKEEPER-311) Changes in hedwig client api to support JMS spec

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mridul Muralidharan updated BOOKKEEPER-311: --- Attachment: hedwig-client-consume.patch.1 > Changes in hedwig client

[jira] [Updated] (BOOKKEEPER-311) Changes in hedwig client api to support JMS spec

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mridul Muralidharan updated BOOKKEEPER-311: --- Attachment: hedwig-client-publish.patch.1 > Changes in hedwig client

[jira] [Updated] (BOOKKEEPER-312) Implementation of JMS provider

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-312?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mridul Muralidharan updated BOOKKEEPER-312: --- Attachment: hedwig-client-jms.patch.2 Changes to implementation in line w

[jira] [Updated] (BOOKKEEPER-310) Changes in hedwig server to support JMS spec

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-310?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mridul Muralidharan updated BOOKKEEPER-310: --- Attachment: hedwig-server.patch.1 Very minor changes to server compared t

[jira] [Updated] (BOOKKEEPER-309) Protocol changes in hedwig to support JMS spec

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mridul Muralidharan updated BOOKKEEPER-309: --- Attachment: hedwig-protocol.patch.1 Protocol changes for adding metadata.

[jira] [Commented] (BOOKKEEPER-308) Add support for JMS implementation for hedwig

2012-07-05 Thread Mridul Muralidharan (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407615#comment-13407615 ] Mridul Muralidharan commented on BOOKKEEPER-308: Have attached patches

[jira] [Commented] (BOOKKEEPER-327) System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407652#comment-13407652 ] Sijie Guo commented on BOOKKEEPER-327: -- ah, I got the reason why it throws ArrayI

[jira] [Commented] (BOOKKEEPER-309) Protocol changes in hedwig to support JMS spec

2012-07-05 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407657#comment-13407657 ] Sijie Guo commented on BOOKKEEPER-309: -- > Protocol changes for adding metadata.

[jira] [Commented] (BOOKKEEPER-327) System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Uma Maheswara Rao G (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407662#comment-13407662 ] Uma Maheswara Rao G commented on BOOKKEEPER-327: Oops, I missed it. el

[jira] [Updated] (BOOKKEEPER-327) System.currentTimeMillis usage in BookKeeper

2012-07-05 Thread Rakesh R (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rakesh R updated BOOKKEEPER-327: Affects Version/s: 4.0.0 4.1.0 > System.currentTimeMillis usage in B