atmosphere
  1. atmosphere
  2. ATMOSPHERE-79

parallel changes overwritten in broadcastercache

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6.1
    • Fix Version/s: 0.7
    • Component/s: None
    • Labels:
      None

      Description

      2 writes in very close succession cause the BroadcasterCache to become corrupt, so that when writing two messages, I see the first message twice.

        Activity

        Hide
        jfarcand added a comment -

        Salut,

        the source is here:

        https://github.com/Atmosphere/atmosphere

        It contains the fix for you. Try it and let me know by downloading the latest 0.7-SNAPSHOT. As your last questions, can you share a test cases?

        Thanks!

        – Jeanfrancois

        Show
        jfarcand added a comment - Salut, the source is here: https://github.com/Atmosphere/atmosphere It contains the fix for you. Try it and let me know by downloading the latest 0.7-SNAPSHOT. As your last questions, can you share a test cases? Thanks! – Jeanfrancois
        Hide
        jfarcand added a comment -

        Just te be sure, which exact version of Atmosphere are you using? I can see some issue if the BroadcasterConfig executorServices size is larger than 1 (I will fix that).

        Show
        jfarcand added a comment - Just te be sure, which exact version of Atmosphere are you using? I can see some issue if the BroadcasterConfig executorServices size is larger than 1 (I will fix that).
        Hide
        jfarcand added a comment -

        OK, I can't fully test it right now but I've pushed a fix. Make sure you download the latest 0.7-SNAPSHOT

        Show
        jfarcand added a comment - OK, I can't fully test it right now but I've pushed a fix. Make sure you download the latest 0.7-SNAPSHOT
        Hide
        mrbeatle added a comment -

        I was using 0.6.2.

        Retested with 0.7-SNAPSHOT. The problem with a message being duplicated or lost is now fixed.

        Show
        mrbeatle added a comment - I was using 0.6.2. Retested with 0.7-SNAPSHOT. The problem with a message being duplicated or lost is now fixed.
        Hide
        jfarcand added a comment -

        Thanks for the feedback. Marking the issue as closed.

        Show
        jfarcand added a comment - Thanks for the feedback. Marking the issue as closed.

          People

          • Assignee:
            Unassigned
            Reporter:
            mrbeatle
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: