torwig commented on code in PR #278:
URL: 
https://github.com/apache/kvrocks-controller/pull/278#discussion_r2000584916


##########
store/engine/engine_inmemory.go:
##########
@@ -75,7 +75,7 @@ func (m *Mock) List(_ context.Context, prefix string) 
([]Entry, error) {
        m.mu.Lock()
        defer m.mu.Unlock()
 
-       exists := make(map[string]bool, 0)
+       exists := map[string]bool{}

Review Comment:
   @Jitmisra Are you sure this change fixes everything you described? From my 
perspective, both ways to declare a map are the same and the result will be the 
same - an empty initialized map.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to