关于表达式缓存的使用优化（避免一些Surprise的感觉）

[oldratlee]

下面的使用方式有 Surprise的感觉：

• 调用了AviatorEvaluatorInstance的useLRUExpressionCache（use*ExpressionCache）方法，
  但不会生效表达式缓存，有些surprise。
• 还需要在执行（类execute方法）时，传递上cached参数为true，才生效表达式缓存。
  • 在执行时，设置『表达式是否缓存』，更显意，是好的。 :")
  • 在执行时，感觉可能有些繁琐； 因为执行方法可能会在不同地方多次执行。
  • 如果『表达式是否缓存』能在AviatorEvaluatorInstance上有个缺省设置：
    • 可以避免上面的『感觉繁琐』
    • 执行时的『表达式是否缓存』设置，可以显意覆盖AviatorEvaluatorInstance级的设置。
• 详见下面的show case代码。

上面可能问题的解决方法（API设计）是不是 可以下面的做法：

1. 在AviatorEvaluatorInstance级，可以（缺省）设置『表达式是否缓存』
2. useLRUExpressionCache/use*ExpressionCache 方法，隐含了AviatorEvaluatorInstance级『表达式要缓存』
   • 顾名思意，符合方法命名（use*ExpressionCache）的直觉，没有Surprise。
   • 这里要注意，向后兼容 的问题。
3. 考虑AviatorEvaluatorInstance级默认开启『表达式要缓存』？ @killme2008
   • 对于大流量线上产品，动态编译出来的Class回收慢会压跨应用。
   • 但默认开启如何设置缓存大小是个难题？😂
   • 有 向后兼容 的问题

如果上面的思路/解决方法觉得OK，我可以试着给个PR。:")

---

public class ExpressionCacheBehaviorSurpriseTest {
    @Test
    public void call_useLRUExpressionCache_butExpressionNotCached__SURPRISE() {
        final Map<String, Object> context = new HashMap<>();
        context.put("var1", 10);
        context.put("var2", 20);
        final String expr = "var1 < 100 && var2 < 200";

        // --> Have called *use*LRUExpressionCache
        final AviatorEvaluatorInstance instance =
                AviatorEvaluator.newInstance().useLRUExpressionCache(100);
        final Boolean result = (Boolean) instance.execute(expr, context);
        assertTrue(result);
        // --> But expression Not cached. SURPRISE!
        assertFalse(instance.isExpressionCached(expr));
    }

    @Test
    public void shouldCall_useLRUExpressionCache_and_passCachedParameterToExecute_then_ExpressionWillCache() {
        final Map<String, Object> context = new HashMap<>();
        context.put("var1", 10);
        context.put("var2", 20);
        final String expr = "var1 < 100 && var2 < 200";

        // --> Have called *use*LRUExpressionCache
        final AviatorEvaluatorInstance instance =
                AviatorEvaluator.newInstance().useLRUExpressionCache(100);
        // --> And pass cached parameter to execute method
        final Boolean result = (Boolean) instance.execute(expr, context, true);
        assertTrue(result);
        // --> Then expression will cache
        assertTrue(instance.isExpressionCached(expr));
    }
}

PS: show case source file

相关资料

• Aviator官方文档：最佳实践 - 使用编译缓存模式
  • AviatorEvaluatorInstance#execute 时设置 cached参数为 true
  • 可以设置使用 LRU Cache

[killme2008]

最终决定引入一个属性，类似 AviatorEvaluatorInstance#setExpressionCaching(boolean) 来设置是否开启全局缓存。

[killme2008]

5.2.2 发布 https://github.com/killme2008/aviatorscript/releases/tag/aviator-5.2.2