Skip to content

Commit 97c2b02

Browse files
authored
Use max_records_per_batch=500 as default (#1031)
This PR should stabilize integration tests a bit more. There are some slight hints that this may get queries done from integration tests unstuck: <img width="1431" alt="image" src="https://github.com/databrickslabs/ucx/assets/259697/f2833640-05b2-43f9-9c25-df92696742d2">
1 parent f6bd734 commit 97c2b02

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/databricks/labs/ucx/framework/crawlers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ def _api_error_from_message(error_message: str) -> DatabricksError:
123123

124124

125125
class StatementExecutionBackend(SqlBackend):
126-
def __init__(self, ws: WorkspaceClient, warehouse_id, *, max_records_per_batch: int = 1000):
126+
def __init__(self, ws: WorkspaceClient, warehouse_id, *, max_records_per_batch: int = 500):
127127
self._sql = StatementExecutionExt(ws)
128128
self._warehouse_id = warehouse_id
129129
self._max_records_per_batch = max_records_per_batch

0 commit comments

Comments
 (0)