[GH-ISSUE #571] [bug] BatchIsExist and BatchIsPut may have errors. #190

New issue

Closed

opened 2026-10-29 21:10:41 +00:00 by bsakaguchi · 2 comments

bsakaguchi commented

2026-10-29 21:10:41 +00:00

Owner

Originally created by @jeremyzhang866 on GitHub (Jun 30, 2025).
Original GitHub issue: https://github.com/kvcache-ai/Mooncake/issues/571

when you test client_integration_test.cpp
ASSERT_EQ(test_client_->BatchIsExist(keys, exist_results), ErrorCode::OK);

[ OK ] ClientIntegrationTest.BatchPutGetOperations (24 ms)
[ RUN ] ClientIntegrationTest.BatchIsExistOperations

but log show that: means that BatchIsExist is error?!
I0701 00:37:27.737293 272358 scoped_vlog_timer.h:75] MasterClient::BatchExistKey response: {"exist_responses":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704]}, latency=820us

Originally created by @jeremyzhang866 on GitHub (Jun 30, 2025). Original GitHub issue: https://github.com/kvcache-ai/Mooncake/issues/571 when you test client_integration_test.cpp `ASSERT_EQ(test_client_->BatchIsExist(keys, exist_results), ErrorCode::OK);` [ OK ] ClientIntegrationTest.BatchPutGetOperations (24 ms) [ RUN ] ClientIntegrationTest.BatchIsExistOperations but log show that: means that BatchIsExist is error?! `I0701 00:37:27.737293 272358 scoped_vlog_timer.h:75] MasterClient::BatchExistKey response: {"exist_responses":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704,-704]}, latency=820us`

bsakaguchi closed this issue

2026-10-29 21:10:41 +00:00

bsakaguchi commented

2026-10-29 21:10:42 +00:00

Author

Owner

@jeremyzhang866 commented on GitHub (Jun 30, 2025):

@xiaguan PTAL. THANKS

@jeremyzhang866 commented on GitHub (Jun 30, 2025): @xiaguan PTAL. THANKS

bsakaguchi commented

2026-10-29 21:10:42 +00:00

Author

Owner

@xiaguan commented on GitHub (Jul 1, 2025):

Before PR #562, we did use OBJECT_NOT_FOUND as the error code in Exist. After PR #562, that issue should no longer happen. The test is intentionally checking cases where some objects exist and some don’t, so the logs look as expected.

@xiaguan commented on GitHub (Jul 1, 2025): Before PR #562, we did use `OBJECT_NOT_FOUND` as the error code in Exist. After PR #562, that issue should no longer happen. The test is intentionally checking cases where some objects exist and some don’t, so the logs look as expected.

bsakaguchi referenced this issue

2026-10-29 21:26:56 +00:00

[PR #190] [MERGED] [Enhance] Add an RPATH that takes precedence over LD_LIBRARY_PATH and other paths in the wheel package #1381