Skip to content

[CLIENT-3947] Deprecate meta["ttl"] parameter in favor of setting ttl as a policy option instead#880

Draft
juliannguyen4 wants to merge 48 commits intodevfrom
CLIENT-3947-deprecate-meta-parameter-for-multiple-client-commands
Draft

[CLIENT-3947] Deprecate meta["ttl"] parameter in favor of setting ttl as a policy option instead#880
juliannguyen4 wants to merge 48 commits intodevfrom
CLIENT-3947-deprecate-meta-parameter-for-multiple-client-commands

Conversation

@juliannguyen4
Copy link
Collaborator

@juliannguyen4 juliannguyen4 commented Dec 5, 2025

This affects all the aerospike.Client methods that accept a metadata dictionary parameter:

  • append()
  • increment()
  • operate()
  • operate_ordered()
  • prepend()
  • put()
  • remove()
  • remove_bin()
  • touch()

As well as BatchRecords Write and Read.

Extra Changes

  • validate_keys: allow "read_mode_ap", "max_threads", and "thread_pool_size" as valid keys in the client config dictionary
  • Fix test that checks if a valid client config option raises a ParamError

Documentation

TODO

  • Update code examples that use meta parameter

@codecov-commenter
Copy link

codecov-commenter commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.96%. Comparing base (b3a3aa4) to head (1a8f297).
⚠️ Report is 7 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #880      +/-   ##
==========================================
+ Coverage   83.53%   83.96%   +0.42%     
==========================================
  Files          99       99              
  Lines       14392    14409      +17     
==========================================
+ Hits        12022    12098      +76     
+ Misses       2370     2311      -59     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@juliannguyen4 juliannguyen4 changed the title [CLIENT-3947] Deprecate meta parameter for multiple client commands [CLIENT-3947] Deprecate meta["ttl"] parameter for multiple client commands Feb 25, 2026
@juliannguyen4 juliannguyen4 changed the title [CLIENT-3947] Deprecate meta["ttl"] parameter for multiple client commands [CLIENT-3947] Deprecate meta["ttl"] parameter in favor of setting ttl as a policy option instead Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants