6e3f1a1194
* Upgrade syntax with pyupgrade --py38-plus Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> * Convert to f-strings with flynt Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> * Format with Black Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> * Remove redundant mock backport dependency Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> * isort imports Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> * Add changelog entry Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> --------- Signed-off-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
169 lines
6.0 KiB
Python
169 lines
6.0 KiB
Python
# SPDX-License-Identifier: Apache-2.0
|
|
#
|
|
# The OpenSearch Contributors require contributions made to
|
|
# this file be licensed under the Apache-2.0 license or a
|
|
# compatible open source license.
|
|
#
|
|
# Modifications Copyright OpenSearch Contributors. See
|
|
# GitHub history for details.
|
|
#
|
|
# Licensed to Elasticsearch B.V. under one or more contributor
|
|
# license agreements. See the NOTICE file distributed with
|
|
# this work for additional information regarding copyright
|
|
# ownership. Elasticsearch B.V. licenses this file to you under
|
|
# the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing,
|
|
# software distributed under the License is distributed on an
|
|
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
# KIND, either express or implied. See the License for the
|
|
# specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
|
|
import time
|
|
from typing import Any
|
|
|
|
from opensearchpy.connection import Connection
|
|
from opensearchpy.connection_pool import (
|
|
ConnectionPool,
|
|
DummyConnectionPool,
|
|
RoundRobinSelector,
|
|
)
|
|
from opensearchpy.exceptions import ImproperlyConfigured
|
|
|
|
from .test_cases import TestCase
|
|
|
|
|
|
class TestConnectionPool(TestCase):
|
|
def test_dummy_cp_raises_exception_on_more_connections(self) -> None:
|
|
self.assertRaises(ImproperlyConfigured, DummyConnectionPool, [])
|
|
self.assertRaises(
|
|
ImproperlyConfigured, DummyConnectionPool, [object(), object()]
|
|
)
|
|
|
|
def test_raises_exception_when_no_connections_defined(self) -> None:
|
|
self.assertRaises(ImproperlyConfigured, ConnectionPool, [])
|
|
|
|
def test_default_round_robin(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
|
|
connections = set()
|
|
for _ in range(100):
|
|
connections.add(pool.get_connection())
|
|
self.assertEqual(connections, set(range(100)))
|
|
|
|
def test_disable_shuffling(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)], randomize_hosts=False)
|
|
|
|
connections = []
|
|
for _ in range(100):
|
|
connections.append(pool.get_connection())
|
|
self.assertEqual(connections, list(range(100)))
|
|
|
|
def test_selectors_have_access_to_connection_opts(self) -> None:
|
|
class MySelector(RoundRobinSelector):
|
|
def select(self, connections: Any) -> Any:
|
|
return self.connection_opts[super().select(connections)]["actual"]
|
|
|
|
pool = ConnectionPool(
|
|
[(x, {"actual": x * x}) for x in range(100)],
|
|
selector_class=MySelector,
|
|
randomize_hosts=False,
|
|
)
|
|
|
|
connections = []
|
|
for _ in range(100):
|
|
connections.append(pool.get_connection())
|
|
self.assertEqual(connections, [x * x for x in range(100)])
|
|
|
|
def test_dead_nodes_are_removed_from_active_connections(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
|
|
now = time.time()
|
|
pool.mark_dead(42, now=now)
|
|
self.assertEqual(99, len(pool.connections))
|
|
self.assertEqual(1, pool.dead.qsize())
|
|
self.assertEqual((now + 60, 42), pool.dead.get())
|
|
|
|
def test_connection_is_skipped_when_dead(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(2)])
|
|
pool.mark_dead(0)
|
|
|
|
self.assertEqual(
|
|
[1, 1, 1],
|
|
[pool.get_connection(), pool.get_connection(), pool.get_connection()],
|
|
)
|
|
|
|
def test_new_connection_is_not_marked_dead(self) -> None:
|
|
# Create 10 connections
|
|
pool = ConnectionPool([(Connection(), {}) for _ in range(10)])
|
|
|
|
# Pass in a new connection that is not in the pool to mark as dead
|
|
new_connection = Connection()
|
|
pool.mark_dead(new_connection)
|
|
|
|
# Nothing should be marked dead
|
|
self.assertEqual(0, len(pool.dead_count))
|
|
|
|
def test_connection_is_forcibly_resurrected_when_no_live_ones_are_available(
|
|
self,
|
|
) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(2)])
|
|
pool.dead_count[0] = 1
|
|
pool.mark_dead(0) # failed twice, longer timeout
|
|
pool.mark_dead(1) # failed the first time, first to be resurrected
|
|
|
|
self.assertEqual([], pool.connections)
|
|
self.assertEqual(1, pool.get_connection())
|
|
self.assertEqual([1], pool.connections)
|
|
|
|
def test_connection_is_resurrected_after_its_timeout(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
|
|
now = time.time()
|
|
pool.mark_dead(42, now=now - 61)
|
|
pool.get_connection()
|
|
self.assertEqual(42, pool.connections[-1])
|
|
self.assertEqual(100, len(pool.connections))
|
|
|
|
def test_force_resurrect_always_returns_a_connection(self) -> None:
|
|
pool = ConnectionPool([(0, {})])
|
|
|
|
pool.connections = []
|
|
self.assertEqual(0, pool.get_connection())
|
|
self.assertEqual([], pool.connections)
|
|
self.assertTrue(pool.dead.empty())
|
|
|
|
def test_already_failed_connection_has_longer_timeout(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
now = time.time()
|
|
pool.dead_count[42] = 2
|
|
pool.mark_dead(42, now=now)
|
|
|
|
self.assertEqual(3, pool.dead_count[42])
|
|
self.assertEqual((now + 4 * 60, 42), pool.dead.get())
|
|
|
|
def test_timeout_for_failed_connections_is_limitted(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
now = time.time()
|
|
pool.dead_count[42] = 245
|
|
pool.mark_dead(42, now=now)
|
|
|
|
self.assertEqual(246, pool.dead_count[42])
|
|
self.assertEqual((now + 32 * 60, 42), pool.dead.get())
|
|
|
|
def test_dead_count_is_wiped_clean_for_connection_if_marked_live(self) -> None:
|
|
pool = ConnectionPool([(x, {}) for x in range(100)])
|
|
now = time.time()
|
|
pool.dead_count[42] = 2
|
|
pool.mark_dead(42, now=now)
|
|
|
|
self.assertEqual(3, pool.dead_count[42])
|
|
pool.mark_live(42)
|
|
self.assertNotIn(42, pool.dead_count)
|