- Function Modification: The
write_time_detailfunction has been refactored intomerge_time_detailto modify the behavior of merging time details instead of overwriting them. Themerge_time_detailfunction now adds new values to the existing ones, preserving the data and allowing for cumulative updates, which ensures more accurate tracking of time metrics. - Usage of New Function: All instances where
write_time_detailwas called have been updated to usemerge_time_detail, including in thesrc/coprocessor/endpoint.rs,src/server/service/kv.rs,src/storage/txn/tracker.rs, and test files. The modification ensures consistency across the codebase by merging time details rather than replacing them. - Test Coverage: A new test,
test_select_time_details, has been added intests/integrations/coprocessor/test_select.rsto validate the proper functioning of time detail merging. The test checks that theprocess_wall_time_nsfield is not zero, ensuring the correct time metrics are being tracked and merged. - Backward Compatibility: The changes do not affect any external functionality or break compatibility. The merging of time details is backward-compatible, as it preserves existing values and adds new ones, which makes the system more flexible for future extensions.
- Code Consistency: The naming convention and function signature have been aligned with existing code practices, making the codebase more consistent and easier to maintain.