The check for connection.isClosed() should come before the check for previous generation. If a connection is closed, the generation will be incremented when that happens, so the check for connection.isClosed() will never be reached.