[Bug]: kv returns more entries than wanted #213
Assignees
Comments
congqixia
added a commit
to congqixia/birdwatcher
that referenced
this issue
Nov 17, 2023
Since `path.Join` will remove tailing slash, the newly added kv implementation will return some other kv values when prefix need this tailing slash. Add a util function to keep this tailing slash and replace all `path.Join` in kv impl with it Fix milvus-io#213 Signed-off-by: Congqi Xia <[email protected]>
sre-ci-robot
pushed a commit
that referenced
this issue
Nov 18, 2023
Since `path.Join` will remove tailing slash, the newly added kv implementation will return some other kv values when prefix need this tailing slash. Add a util function to keep this tailing slash and replace all `path.Join` in kv impl with it Fix #213 Signed-off-by: Congqi Xia <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current Behavior
Show segment may treat stats log entry as segment
Get with prefix with tailing '/', this '/' will be ignored
Expected Behavior
tailing '/' cannot be ignore for kv operations
Steps To Reproduce
No response
Environment
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: