Skip to content

Commit

Permalink
Add retries on RPC token bucket limit has been exceeded (#395)
Browse files Browse the repository at this point in the history
## Changes

Another case when the platform is not sending a proper HTTP 429
response. This time for the SCIM user list:
<img width="925" alt="image"
src="https://github.com/databricks/databricks-sdk-py/assets/259697/c6159844-949b-4746-8750-7e7125056493">


## Tests
<!-- 
How is this tested? Please see the checklist below and also describe any
other relevant tests
-->

- [ ] `make test` run locally
- [ ] `make fmt` applied
- [ ] relevant integration tests applied

---------

Signed-off-by: Serge Smertin <259697+nfx@users.noreply.github.com>
  • Loading branch information
nfx committed Oct 10, 2023
1 parent 26e41fe commit afeff49
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion databricks/sdk/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -1097,7 +1097,8 @@ def _is_retryable(err: BaseException) -> Optional[str]:
"com.databricks.backend.manager.util.UnknownWorkerEnvironmentException",
"does not have any associated worker environments", "There is no worker environment with id",
"Unknown worker environment", "ClusterNotReadyException", "Unexpected error",
"Please try again later or try a faster operation."
"Please try again later or try a faster operation.",
"RPC token bucket limit has been exceeded",
]
for substring in transient_error_string_matches:
if substring not in message:
Expand Down

0 comments on commit afeff49

Please sign in to comment.