indigo icon indicating copy to clipboard operation
indigo copied to clipboard

Batch performance suggestions

Open davesmith00000 opened this issue 1 year ago • 3 comments

You might get a quick and easy performance boost by making it a sealed abstract class instead of a sealed trait. That would make the access to the private lazy val monomorphic.

So says @sjrd.

Try it. Benchmark it. Sounds like a fun task to me. :)

davesmith00000 avatar Feb 06 '24 08:02 davesmith00000

Benchmarks, before:

image

image

davesmith00000 avatar Feb 25 '24 17:02 davesmith00000

Benchmarks, after:

image

image

davesmith00000 avatar Feb 25 '24 17:02 davesmith00000

I'm calling that inconclusive. I'll have to re-run with more iterations to get more stable numbers. Notice that List is jumping about and it shouldn't be.

davesmith00000 avatar Feb 25 '24 18:02 davesmith00000