Skip to content

fix(deps): update module github.com/aerospike/aerospike-client-go/v7 to v8 #38978

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 25, 2025

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
github.com/aerospike/aerospike-client-go/v7 v7.9.0 -> v8.2.0 age adoption passing confidence

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

⚠️ MAJOR VERSION UPDATE ⚠️ - please manually update this package


Release Notes

aerospike/aerospike-client-go (github.com/aerospike/aerospike-client-go/v7)

v8.2.0

Compare Source

  • Fixes

    • [CLIENT-3379] Setting TotalTimeout to 0 causes Scan / Query to fail (timeout).
    • [CLIENT-3376] BatchRead on Non-Existing namespace with transaction policy does not throw expected error.
    • [CLIENT-3353] Insufficient Error Messages from MRT commit failure.
    • [CLIENT-3374] Fix panic when trying to commit a MRT with no read and write transaction.
  • Improvements

    • [CLIENT-3361] Update documentation for RecordsPerSecond scan/query policy.

v8.1.0

Compare Source

  • New Features

    • [CLIENT-3334] Support old PHP7 client encoded boolean and null values.
  • Fixes

    • [CLIENT-3347] MRT error occurrence varies depending on the number of records. PR #​465, thanks to Yevgeny Rizhkov
    • [CLIENT-3348] Parsing error during node rack update.
    • Fix nil check for error in executeIter method. PR #​465, thanks to Oleksii
  • Improvements

    • [CLIENT-3349] Reject BinNames > 15 bytes on client-side.

v8.0.1

Compare Source

  • Fixes
    • [CLIENT-3305] New key struct causes incorrect generation increment in MRT.
    • [CLIENT-3326] Fix UDF can read a record with an expired transaction.

v8.0.0

Compare Source

  • New Features

    • [CLIENT-3257] Implement iterator support for Recordset.
  • Fixes

    • [CLIENT-3275] Return an error when Transaction commit called, but transaction was already aborted.
    • [CLIENT-3295] Duplicate parsing of fields in RecordParser.ParseRecord.
    • [CLIENT-3261] New OnLockingOnly attribute for various write policies to write only if provisional does not exist.
    • [CLIENT-3227] Clarify that BasePolicy.UseCompression requires Enterprise Edition.
    • [CLIENT-3260] Add MRT_ALREADY_LOCKED and MRT_MONITOR_EXISTS error codes.
    • [CLIENT-3292][CLIENT-3293] Update dependencies due to Snyk CVE reports.
    • [CLIENT-3274] Fix Transaction related client exception inconsistency between clients.
    • [CLIENT-3207] Do not close/delete Transaction monitor record on abort/commit when a write command in that Transaction fails and is inDoubt.
    • [CLIENT-3283] Remove the txn instance from the copied write policy when adding Transaction monitor keys.
    • [CLIENT-3270] Normalize API reference language with server 8.0 site docs.
    • [CLIENT-3258] Transaction commit returns 'Failed to commit one or more record versions' error.

Configuration

📅 Schedule: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot added the dependency-major-update Indicates a dependency major version bump label Mar 25, 2025
@renovate renovate bot requested a review from a team as a code owner March 25, 2025 21:31
@renovate renovate bot requested a review from jpkrohling March 25, 2025 21:31
@github-actions github-actions bot requested a review from antonblock March 25, 2025 21:31
@atoulme atoulme added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Mar 25, 2025
@atoulme atoulme closed this Mar 26, 2025
@atoulme atoulme reopened this Mar 26, 2025
@atoulme atoulme added dependencies Pull requests that update a dependency file and removed dependency-major-update Indicates a dependency major version bump labels Mar 26, 2025
@crobert-1
Copy link
Member

I went through and did a lot of find+replace, even on some files that were generated by mockery. I think this should work on the generated code as it's just a dependency bump, but if anything is broken, or should be done differently, we'll need to investigate.

Review from @antonblock would be much appreciated to make sure I didn't break anything.

@crobert-1
Copy link
Member

Looks like there's a breaking change that needs accounted for:

Error: /home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/receiver/aerospikereceiver/cluster/node.go:38:9: cannot use as.NewConnection(policy, host) (value of type *aerospike.Connection) as asconn value in return statement: *aerospike.Connection does not implement asconn (wrong type for method SetTimeout)
		have SetTimeout(time.Duration, time.Duration) aerospike.Error
		want SetTimeout(time.Time, time.Duration) aerospike.Error

Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file receiver/aerospike Skip Changelog PRs that do not require a CHANGELOG.md entry Stale waiting-for-code-owners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants