1 2 Previous Next 18 Replies Latest reply: Feb 9, 2012 6:01 PM by Clebert Suconic Go to original post RSS
  • 15. Re: Notes on trunk
    Clebert Suconic Master

    I just committed the changes.. but there are a few issues to be solved now...

     

    Instead of creating a new branch.. I decided to just commit it on trunk.

     

    There was one change on blocking at failover on Branch_2_2, and due to the fact that all sessions are now closed on tests. there are a few FailoverTests hanging.

     

     

    First one I'm investigating is StaticClusterWithBackupFailoverTest (all all the other siblins tests that share the same base class).

     

    @Francisco: if you could help me investigate this test.

     

     

    Also: one thing I would like to eliminate from the code base is the CloseRunnables on the ClientSessionImpl. Is that really needed?

  • 16. Re: Notes on trunk
    Clebert Suconic Master

    I believe that there's something that was broken even before...

     

    I have seen a few failover tests that were changed to stop backup and start live manually (instead of letting the backup to be activated based on locks).

     

     

    This is probably related here.

  • 17. Re: Notes on trunk
    Clebert Suconic Master

    StaticClusterWithBackupFailoverTest was being ignored on 2.2

     

    The test fails because the server is sending a disconnect and the topology is being removed on client.

     

    I have had a hard time understanding the test but I'm on the right track now.

  • 18. Re: Notes on trunk
    Clebert Suconic Master

    The test is failing because of the lockdown we have on close. It's a more general issue on trunk.

     

    We should talk about it on IRC this friday.

1 2 Previous Next