Skip to content

Commit

Permalink
[CLIENT-3205] Multi record transactions: support correct server error…
Browse files Browse the repository at this point in the history
… codes for MRTs (#701)

- [CLIENT-3199] Do not close/delete MRT monitor record on abort/commit when a write command in that MRT fails and is in doubt
- [CLIENT-3212] Raise TranactionFailed exception instead of BatchFailed exception when batch MRT verify fails.
- Always unlock macOS keychain on self-hosted macOS runners to prevent workflow from failing
  • Loading branch information
juliannguyen4 authored Dec 9, 2024
1 parent 0ab340b commit 11d64db
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/build-wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,8 @@ jobs:
ref: ${{ env.COMMIT_SHA_TO_BUILD_AND_TEST }}

# Need to be able to save Docker Hub credentials to keychain
- if: ${{ inputs.platform-tag == 'macosx_arm64' && inputs.use-server-rc }}
# Unable to do this via the ansible script for self hosted mac m1 runners
- if: ${{ inputs.platform-tag == 'macosx_arm64' }}
run: security unlock-keychain -p ${{ secrets.MAC_M1_SELF_HOSTED_RUNNER_PW }}

- uses: ./.github/actions/run-ee-server
Expand Down
2 changes: 2 additions & 0 deletions src/main/exception.c
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,8 @@ struct exception_def exception_defs[] = {
AEROSPIKE_ERR_ASYNC_CONNECTION, NULL),
EXCEPTION_DEF("ClientAbortError", CLIENT_ERR_EXCEPTION_NAME,
AEROSPIKE_ERR_CLIENT_ABORT, NULL),
EXCEPTION_DEF("TranactionFailed", CLIENT_ERR_EXCEPTION_NAME,
AEROSPIKE_TXN_FAILED, NULL),
// Server errors
EXCEPTION_DEF("InvalidRequest", SERVER_ERR_EXCEPTION_NAME,
AEROSPIKE_ERR_REQUEST_INVALID, NULL),
Expand Down

0 comments on commit 11d64db

Please sign in to comment.