Alan-Jowett / CoPilot-For-Consensus / 20724598550

81%

push

github

Implement client-side $lookup for Azure Cosmos DB aggregation and integrate $in operator support (#738)

* Initial plan

* Implement client-side $lookup for Azure Cosmos aggregation

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add clarifying comments for $eq operator array handling

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback: add validation and prevent document mutation

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address PR review: nested field support, validation, error handling, and shallow copy issues

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Improve error messages and code clarity in validation logic

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Merge main branch to resolve conflicts and integrate $in operator support

- Resolved conflict in azure_cosmos_document_store.py by keeping refactored structure
- Integrated $in operator support from main into both query_documents and aggregate_documents
- Updated LIMIT syntax to OFFSET 0 LIMIT as per Cosmos DB requirements
- Fixed exception handling to properly re-raise DocumentStoreError without wrapping
- Added validation for non-operator keys in filter dictionaries
- All 58 Azure Cosmos DB document store tests pass

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix shared mutable state in $lookup and remove duplicate test

- Use deep copy for foreign documents in $lookup to prevent shared mutable state
- Multiple documents joining with same foreign key now get independent copies
- Removed duplicate test_aggregate_documents_invalid_limit (was at line 830)
- All 58 Azure Cosmos DB tests pass

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Raise DocumentStoreError for $lookup validation failures

- Changed from logging errors and returni... (continued)

141 of 170 new or added lines in 1 file covered. (82.94%)

6207 of 8077 relevant lines covered (76.85%)

0.86 hits per line