Skip to content

Commit 32548a6

Browse files
mykauldkropachev
authored andcommitted
Fix unfilled format string in add_execution_profile timeout message
The error message at Cluster.add_execution_profile() had an unfilled %s placeholder: 'Failed to create all new connection pools in the %ss timeout.' The pool_wait_timeout value was never interpolated into the string, so users would see a literal '%s' instead of the actual timeout value. Signed-off-by: Yaniv Kaul <yaniv.kaul@scylladb.com>
1 parent 5cd0158 commit 32548a6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

cassandra/cluster.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1683,7 +1683,7 @@ def add_execution_profile(self, name, profile, pool_wait_timeout=5):
16831683
futures.update(session.update_created_pools())
16841684
_, not_done = wait_futures(futures, pool_wait_timeout)
16851685
if not_done:
1686-
raise OperationTimedOut("Failed to create all new connection pools in the %ss timeout.")
1686+
raise OperationTimedOut("Failed to create all new connection pools in the %ss timeout." % pool_wait_timeout)
16871687

16881688
def connection_factory(self, endpoint, host_conn = None, *args, **kwargs):
16891689
"""

0 commit comments

Comments
 (0)