Skip to content

[CLIENT-4000] Add support for bin projection.#998

Draft
juliannguyen4 wants to merge 41 commits intodevfrom
CLIENT-4000-query-read-ops-projection
Draft

[CLIENT-4000] Add support for bin projection.#998
juliannguyen4 wants to merge 41 commits intodevfrom
CLIENT-4000-query-read-ops-projection

Conversation

@juliannguyen4
Copy link
Copy Markdown
Collaborator

@juliannguyen4 juliannguyen4 commented Mar 23, 2026

TODO

  • Document extra C client changes
  • Fix potential memory leak in Query.add_ops() from the list of operations passed as an argument
  • Decide what to do with Scan class
  • CLIENT-4046 Change default interval from 60s to 5s (TODO needs to be done manually in Python client)
  • Running against server < 8.1.2 doesn't produce a server error
  • Produce warning if selected bins and operations are added at the same time. Also requires test case Error already produced by C client in dev.
  • [ ]

Extra Changes

  • Provide better error messaging for transactions
  • Address crash caused by TLS socket error handling by checking for WSAEWOULDBLOCK instead of EWOULDBLOCK on Windows.
  • Document that Query.apply() can be used to apply record UDFs
  • Change ConfigProvider's interval parameter default value from 60000 to 5000.

Known Issues (out of scope)

  • Client config option "conn_pools_per_node" is missing

Docs

https://aerospike-python-client--998.org.readthedocs.build/en/998/query.html#overview
https://aerospike-python-client--998.org.readthedocs.build/en/998/query.html#aerospike.Query.add_ops

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.83%. Comparing base (274742f) to head (e7c6c15).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
src/main/query/select.c 75.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #998   +/-   ##
=======================================
  Coverage   83.83%   83.83%           
=======================================
  Files          99       99           
  Lines       14429    14437    +8     
=======================================
+ Hits        12096    12103    +7     
- Misses       2333     2334    +1     

☔ 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.

…ntation parts of the c client to implement ResultRemove()
@juliannguyen4 juliannguyen4 force-pushed the CLIENT-4000-query-read-ops-projection branch from 3eb3836 to 3286a22 Compare March 23, 2026 19:40
…remove the dependency to as_connection directly
…g info about using query.apply() to apply record UDFs.
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