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