search

moosetechnology / Fame

Fame is a meta-modeling framework for executable models.
MIT License
4 stars 10 forks source link

Remove cache of slots per class in FMRelationSlot #21

Open NicolasAnquetil opened 3 years ago

NicolasAnquetil commented 3 years ago

I found a case (PowerBuilder) were this cache was causing bugs Is it really needed ?

jecisc commented 3 years ago

I've added the cache because it cause a great performance regression on MooseQuery on big models. The introduction of the cache improved a lot the performances of Benoit's tooling

badetitou commented 3 years ago

Yep. The cache is super important in my tool.

Is it creates bug, I know that before there was a strategy pattern that allows us to use or not cache

We can do it again maybe? By default use cache ( it is nice for user imo ). And one can choose to not use it.

Are you ok with that @NicolasAnquetil?