FirelyTeam / firely-cql-sdk

BSD 3-Clause "New" or "Revised" License
31 stars 17 forks source link

Performance hit with numerous class initializations (2.0) #575

Closed baseTwo closed 1 month ago

baseTwo commented 2 months ago

This is the same ticket as below, only for the 2.0 branch:

ewoutkramer commented 1 month ago

To improve over the 1.0 implementation, we need to make sure that:

Put otherwise: we should keep the old situation, except for CQL libraries that only contain CQL functions.

ewoutkramer commented 1 month ago

An idea that we discussed was to put the memoization in the CqlContext. This would require you to be smart with the key, since the names of the functions can be aliased etc. We initially used Lazy, since it was faster, and we would need to do perf testing to see whether moving the state to CqlContext is slower.