-
Notifications
You must be signed in to change notification settings - Fork 322
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: delete unused images from images dir #2083
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2083 +/- ##
============================================
- Coverage 75.72% 75.71% -0.02%
Complexity 347 347
============================================
Files 615 615
Lines 116901 116901
Branches 1015 1015
============================================
- Hits 88523 88507 -16
- Misses 28169 28185 +16
Partials 209 209
Continue to review full report at Codecov.
|
@@ -43,10 +43,10 @@ void OnLockedCallback() { call_invoked = true; } | |||
void OnLostCallback() {} | |||
|
|||
TEST_F(DistLockTest, Lock) { | |||
ZkClient client("127.0.0.1:6181", "", 10000, "127.0.0.1:9527", "/rtidb_lock"); | |||
ZkClient client("127.0.0.1:6181", "", 10000, "127.0.0.1:9527", "/openmldb_lock"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @KikiDotPy , please revert zk/dist_lock_test.cc
and zk/zk_client_test.cc
. This task has been done in #2086
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Deleted images requested in issues #2082
images/fedb_arch.png
images/fedb_black.png
images/openmldb_architecture.jpg
images/openmldb_architecture.png