评论者: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
;; 小型地图
user=> (def ys (range 16))
'user/ys
user=> (last ys)
15
user=> (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
user=> (c/bench (transient-zipmap ys ys))
评估次数:18536880次,在308948次调用中的60个样本。
Execution time mean : 3.412992 us
Execution time std-deviation : 81.338284 ns
执行时间下四分位数:3.303888微秒(25%)
执行时间上四分位数:3.545549微秒(97.5%)
nil
`
显然,只要临时变量满足其契约,语义就被保留。
我想我可能没有为这个启动一个ggroup线程,抱歉。