Skip to content

Commit 725f363

Browse files
authored
Revert "apierr: Alias ErrResourceAlreadyExists to ErrAlreadyExists" (#1658)
Reverts #1639
1 parent 06b4272 commit 725f363

3 files changed

Lines changed: 5 additions & 15 deletions

File tree

NEXT_CHANGELOG.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88

99
### Bug Fixes
1010

11-
* Alias `apierr.ErrResourceAlreadyExists` to `apierr.ErrAlreadyExists` so `errors.Is` matches both the gRPC-canonical `ALREADY_EXISTS` and the Databricks-specific `RESOURCE_ALREADY_EXISTS` error codes, regardless of which one the backend returns.
12-
1311
### Documentation
1412

1513
### Internal Changes

apierr/error_mapping.go

Lines changed: 5 additions & 9 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apierr/unwrap_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,7 @@ var testTable = []mapping{
1717
codeMapping(409, "ABORTED", ErrAborted),
1818
codeMapping(409, "ABORTED", ErrResourceConflict),
1919
codeMapping(409, "ALREADY_EXISTS", ErrAlreadyExists),
20-
codeMapping(409, "ALREADY_EXISTS", ErrResourceAlreadyExists),
2120
codeMapping(409, "ALREADY_EXISTS", ErrResourceConflict),
22-
codeMapping(409, "RESOURCE_ALREADY_EXISTS", ErrAlreadyExists),
23-
codeMapping(409, "RESOURCE_ALREADY_EXISTS", ErrResourceAlreadyExists),
24-
codeMapping(409, "RESOURCE_ALREADY_EXISTS", ErrResourceConflict),
2521
statusMapping(429, ErrTooManyRequests),
2622
codeMapping(429, "REQUEST_LIMIT_EXCEEDED", ErrTooManyRequests),
2723
codeMapping(429, "REQUEST_LIMIT_EXCEEDED", ErrRequestLimitExceeded),

0 commit comments

Comments
 (0)