nickva commented on code in PR #5181:
URL: https://github.com/apache/couchdb/pull/5181#discussion_r1720597751
##########
src/couch/test/eunit/couch_key_tree_tests.erl:
##########
@@ -575,5 +575,10 @@ should_not_use_excessive_memory_when_stemming() ->
receive
{'DOWN', Ref, _, _, Res} -> Res
end,
- ?assertEqual(normal, Exit)
+ case erlang:system_info(otp_release) of
+ Ver when Ver =:= "27" ->
+ ?assertEqual(killed, Exit);
Review Comment:
We should see what's happening here. The idea is we don't want stemming to
take too much memory. Using a set based map helped keep the data on the heap,
we were able to keep the max memory size down. It seems in 27 either the
allocation strategy changes, the compiler changed or the memory calculation
size changed. What if we increase it just a bit, maybe 16_000_000 would do it?
--
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]
