[jboss-jira] [JBoss JIRA] (JGRP-1461) SEQUENCER leaks sequence numbers

David Hotham (JIRA) jira-events at lists.jboss.org
Thu May 10 05:18:18 EDT 2012


    [ https://issues.jboss.org/browse/JGRP-1461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12691731#comment-12691731 ] 

David Hotham commented on JGRP-1461:
------------------------------------

OK, thanks.

It's certainly in my interests to have SEQUENCER in as good shape as possible - if you think I can help, I'd be glad to do so.  I'm an IRC novice I'm afraid; but I'm sure I can figure it out...

Should I be worried by your commit moving SEQUENCER ("needs to be above flow control protocols")?  Per JGRP-1426, I want views to be sequenced too, so that part of my stack goes, from bottom to top: SEQUENCER, GMS, UFC, MFC.  Am I likely to hit trouble by having SEQUENCER below the flow control protocols?
                
> SEQUENCER leaks sequence numbers
> --------------------------------
>
>                 Key: JGRP-1461
>                 URL: https://issues.jboss.org/browse/JGRP-1461
>             Project: JGroups
>          Issue Type: Bug
>    Affects Versions: 3.0.9, 3.1
>            Reporter: David Hotham
>            Assignee: Bela Ban
>             Fix For: 3.0.10, 3.1
>
>
> As noted in JGRP-1458, it looks as though SEQUENCER always thinks that sequence numbers must start at zero.  But a member who joins a group after some messages have already been broadcast may never receive sequence number zero.  So, I think, his SeqnoTable will grow indefinitely.
> Note that this one's mostly through code reading.  I think I'm right; but I'm happy to be told otherwise!
> Pull request to follow shortly.

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

        


More information about the jboss-jira mailing list