Project

General

Profile

Actions

Mejora #752

closed

Implement evaluation cache

Added by Federico Vera almost 7 years ago. Updated over 6 years ago.

Status:
Resuelto
Priority:
Wishlist
Assignee:
Target version:
Start date:
2018-07-10
Due date:
2018-08-01
% Done:

100%

Estimated time:
Dificultad:
Tags:

Description

As of now, every call to Expression#evaluate() reevaluates the whole expression.
This isn't a problem in and of itself, but it's a common error when using the library and creates an annoying performance hit.

  • Find all scenarios where this could fail
  • If an expression contains non-deterministic results, the value should not be cached
  • Cache should be cleaned if variable changes
  • What happens when using setVariables() with maps?
  • Create tests for expression evaluate

PS: We attempted to fix this with documentation and warnings... but still creates a problem. An internal (at least partial) solution shouldn't be hard to accomplish... I think...

Actions

Also available in: Atom PDF