[ZOOKEEPER-1852] ServerCnxnFactory instance is not properly cleanedup

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.4.6, 3.5.0
• Component/s: tests
• Assignee: Rakesh Radhakrishnan
• Reporter: Rakesh Radhakrishnan
• Created: 26/Dec/13 10:20
• Updated: 13/Mar/14 18:17
• Resolved: 03/Jan/14 00:48

Description

ClientBase#createNewServerInstance() - Say the startup of the server fails, this will not initialize 'serverFactory' and will be null. When the flow comes to teardown/shutdown, it will bypass stopping of this server instance due to the following check. This will affect other test case verifications like, jmx check 'JMXEnv#ensureOnly'.

ClientBase#shutdownServerInstance

    static void shutdownServerInstance(ServerCnxnFactory factory,
            String hostPort)
    {
        if (factory != null) {
           //...shutdown logic
        }