flyrain commented on code in PR #72:
URL: https://github.com/apache/polaris-tools/pull/72#discussion_r2558190235
##########
mcp-server/polaris_mcp/server.py:
##########
@@ -59,13 +59,30 @@
"required": ["isError"],
"additionalProperties": True,
}
+DEFAULT_HTTP_RETRIES_TOTAL = 3
+DEFAULT_HTTP_RETRIES_BACKOFF_FACTOR = 0.5
+HTTP_RETRIES_STATUS_FORCELIST = [401, 409, 429, 500, 502, 503, 504]
Review Comment:
It's unlikely now, but still possible that users can commit to a Iceberg
table via MCP, as under the hood, they are IRC endpoints.
For example, we could actually make the MCP more useful by integrating
pyIceberg, which supports appending data.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
