评论由:michalmarczyk发布
嗨 Aaron,
感谢你对此的关注!
根据我所观察到的,这个更改极大地提高了{{zipmap}}在大型图中的性能。对于小型图,有轻微的改进。这里提供两个基本的Criterium基准测试({{transient-zipmap}}在REPL中定义为补丁中所示)
`
==; 大型图==;
user=> (def xs (range 16384))
'user/xs
user=> (last xs)
16383
user=> (c/bench (zipmap xs xs))
评估计数:13920次,共60个样本,每次232次调用。
Execution time mean : 4.329635 ms
Execution time std-deviation : 77.791989 us
执行时间下四分位数:4.215050毫秒(2.5%)
执行时间上四分位数:4.494120毫秒(97.5%)
nil
user=> (c/bench (transient-zipmap xs xs))
评估计数:21180次,共60个样本,每次353次调用。
Execution time mean : 2.818339 ms
Execution time std-deviation : 110.751493 us
执行时间下四分位数:2.618971毫秒(2.5%)
执行时间上四分位数:3.025812毫秒(97.5%)
在60个样本中发现2个异常值(3.3333%)
low-severe 2 (3.3333 %)
异常值带来的方差:25.4675%,方差被异常值中度夸大了
nil
;;; 小图
用户=> (def ys (range 16))
'用户/ys
用户=> (last ys)
15
用户=> (c/bench (zipmap ys ys))
评估计数:16639020,在60个样本中,共有277317次调用。
Execution time mean : 3.803683 us
Execution time std-deviation : 88.431220 ns
执行时间下四分位数:3.638146微秒(25%)
执行时间上四分位数:3.935160微秒(97.5%)
nil
用户=> (c/bench (transient-zipmap ys ys))
评估计数:18536880,在60个样本中,共有308948次调用。
Execution time mean : 3.412992 us
Execution time std-deviation : 81.338284 ns
执行时间下四分位数:3.303888微秒(25%)
执行时间上四分位数:3.545549微秒(97.5%)
nil
`
很明显,只要瞬时对象满足它们的契约,就可以保留语义。
我认为我可能没有启动一个ggroup线程来处理这个问题,抱歉。