Implement KV batch PutMany #1865

Merged
mfreeman451 merged 1 commit from refs/pull/1865/head into 1113-update-armis-integration 2025-06-24 02:52:14 +00:00
mfreeman451 commented 2025-06-24 02:49:48 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #1118
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/1118
Original created: 2025-06-24T02:49:48Z
Original updated: 2025-06-24T02:52:18Z
Original head: carverauto/serviceradar:codex/implement-batch-write-api-for-kv
Original base: 1113-update-armis-integration
Original merged: 2025-06-24T02:52:14Z by @mfreeman451

Summary

  • add PutMany RPC to kv.proto and regenerate protobuf code
  • extend KVService server and NATS store with PutMany handling
  • update sync poller to batch KV writes using PutMany
  • revise KV interfaces and mocks for PutMany support
  • refresh Armis integration tests and sweep result utilities for new APIs

Testing

  • go test ./... (fails: TestArmisIntegration_Fetch_WithUpdaterAndCorrelation, TestDefaultKVWriter_WriteSweepConfig, TestSweepResultsQuery_Pagination in pkg/sync/integrations/armis)

https://chatgpt.com/codex/tasks/task_e_685a0deb86cc83209ded43866c89c806

Imported from GitHub pull request. Original GitHub pull request: #1118 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/1118 Original created: 2025-06-24T02:49:48Z Original updated: 2025-06-24T02:52:18Z Original head: carverauto/serviceradar:codex/implement-batch-write-api-for-kv Original base: 1113-update-armis-integration Original merged: 2025-06-24T02:52:14Z by @mfreeman451 --- ## Summary - add PutMany RPC to kv.proto and regenerate protobuf code - extend KVService server and NATS store with PutMany handling - update sync poller to batch KV writes using PutMany - revise KV interfaces and mocks for PutMany support - refresh Armis integration tests and sweep result utilities for new APIs ## Testing - `go test ./...` *(fails: TestArmisIntegration_Fetch_WithUpdaterAndCorrelation, TestDefaultKVWriter_WriteSweepConfig, TestSweepResultsQuery_Pagination in pkg/sync/integrations/armis)* ------ https://chatgpt.com/codex/tasks/task_e_685a0deb86cc83209ded43866c89c806
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
carverauto/serviceradar!1865
No description provided.