nickva commented on code in PR #5165:
URL: https://github.com/apache/couchdb/pull/5165#discussion_r1702884283
##########
src/couch/src/couch_lru.erl:
##########
@@ -16,33 +16,34 @@
-include("couch_server_int.hrl").
new() ->
- {gb_trees:empty(), dict:new()}.
+ % {gb_trees(UniqueInt -> DbName), #{DbName => UniqueInt}}
+ {gb_trees:empty(), #{}}.
-insert(DbName, {Tree0, Dict0}) ->
+insert(DbName, {Tree0, #{} = Dict0}) ->
Review Comment:
That would work, good idea. I thought about it, but couldn't decide thinking
a "dict" is technically is not incorrect in the general sense. But, yeah it
would be less confusing if the name matched the data type.
--
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]
