mistercrunch commented on code in PR #34290: URL: https://github.com/apache/superset/pull/34290#discussion_r2226862824
########## superset/config.py: ########## @@ -166,6 +166,17 @@ def _try_json_readsha(filepath: str, length: int) -> str | None: NATIVE_FILTER_DEFAULT_ROW_LIMIT = 1000 # max rows retrieved by filter select auto complete FILTER_SELECT_ROW_LIMIT = 10000 + +# SupersetClient HTTP retry configuration +# Controls retry behavior for all HTTP requests made through SupersetClient +# This helps handle transient server errors (like 502 Bad Gateway) automatically +SUPERSET_CLIENT_RETRY_ATTEMPTS = 3 # Maximum number of retry attempts Review Comment: could take either side of that debate, but would a nested object a better idea than multiple config keys here? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
