Skip to content

Add observability for lock poison-recovery and new bulk HNSW build paths #166

Description

@coderabbitai

Summary

Silent lock-recovery and the new bulk-build paths alter runtime behaviour without new logs, metrics, or spans at the decision points.

Required changes

  • Emit a warning/metric or tracing span when poison recovery occurs (e.g. in `DistanceCache`, `RecordingLayer`, `FixedMonotonicClock`, `init_logging`).
  • Add a build-level span/latency metric for the new bulk HNSW constructors (`CpuHnsw::build`, `build_with_edges`, `with_capacity`).

Affected areas

  • `chutoro-core/src/hnsw/distance_cache.rs`
  • `chutoro-core/src/hnsw/cpu/construction.rs`
  • `chutoro-test-support/src/lib.rs`
  • `chutoro-cli/src/logging.rs`
  • `chutoro-core/src/session/clock.rs`

Acceptance criteria

  • Poison-recovery events are observable (log/metric/span).
  • Bulk HNSW build paths emit latency telemetry.

References

  • PR: $pr_url
  • Requested by: @$assignee

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestmediumMedium priority issue

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions