feature: add logging in orchestrator

[yuejiaointel]

Follow up PR on logging
Target:
Adding logging in
orchestrator level:
Functions to add:
experimental_calibrate
search
consolidate
add_points
delete_points/entries

index level:
compact

[Copilot]

Pull Request Overview

This pull request adds a logger parameter to multiple functions within the orchestrator and index modules to enable detailed logging of operations such as calibration, search, consolidation, point addition, and deletion. Key changes include propagating the logger argument through function calls, updating test calls to include logger parameters, and ensuring fallback behavior for logger usage is maintained.

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tests/svs/index/vamana/dynamic_index_2.cpp	Updated test_loop to pass logger to add/delete calls
tests/integration/vamana/index_search.cpp	Propagated logger and introduced log verification checks
include/svs/orchestrators/vamana.h	Added logger parameter to orchestrator APIs
include/svs/orchestrators/manager.h	Updated manager interface to support logger
include/svs/orchestrators/dynamic_vamana.h	Added logger parameter to dynamic operations
... (other files updated similarly)	Logger parameter added and propagated throughout

Comments suppressed due to low confidence (2)

include/svs/orchestrators/vamana.h:95

• Consider updating the inline documentation for the function signature to include information about the new logger parameter and its default behavior.

        const index::vamana::CalibrationParameters& calibration_parameters,

include/svs/index/vamana/dynamic_index.h:707

• It would be beneficial to document the fallback logic (using logger_ if available, otherwise logger) so that users understand how the logger is determined in this context.

        );