Skip to content

Commit 10016ea

Browse files
committed
Add comment about manual tests
Signed-off-by: Jesse Whitehouse <[email protected]>
1 parent baff3d5 commit 10016ea

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/databricks/sql/thrift_backend.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -311,6 +311,10 @@ def attempt_request(attempt):
311311
# The built-in errno package encapsulates OSError codes, which are OS-specific.
312312
# log.info for errors we believe are not unusual or unexpected. log.warn for
313313
# for others like EEXIST, EBADF, ERANGE which are not expected in this context.
314+
#
315+
# I manually tested this retry behaviour using mitmweb and confirmed that
316+
# GetOperationStatus requests are retried when I forced network connection
317+
# interruptions / timeouts / reconnects. See #24 for more info.
314318
# | Debian | Darwin |
315319
info_errs = [ # |--------|--------|
316320
errno.ESHUTDOWN, # | 32 | 32 |

0 commit comments

Comments
 (0)