InvocationTimeout优化

[qubo11]

InvocationTimeoutBootListener类中，对于业务执行前后的Event事件类是否可以统一？

目前开始事件是InvocationBusinessMethodStartEvent，结束事件是InvocationBusinessFinishEvent

在业务方法试行完成，发送EventBus事件的时候，onBusinessMethodFinish/onBusinessFinish都会发送。
是否InvocationBusinessMethodStartEvent和InvocationBusinessMethodFinishEvent成对出现好理解一些？

[wujimin]

这是为了支持异步场景

method start / method finish只表示controller方法执行的开始与结束
对于异步场景，方法结束不等于业务逻辑结束，到了business finished才是业务逻辑结束

每个事件的发布都是有成本的，应该不要发布不必要的事件

[qubo11]

不是这个意思哈。

和异步没有关系，同步也是这样。

我的意思是，invocation.onBusinessMethodStart();发布了InvocationBusinessMethodStartEvent事件，然后InvocationTimeoutBootListener类中对这个事件处理了。invocation.onBusinessMethodFinish();发布了InvocationBusinessMethodFinishEvent事件，但是InvocationTimeoutBootListener类中处理的是invocation.onBusinessFinish();这个方法发布的事件：

我看代码的时候就比较疑惑，为啥InvocationTimeoutBootListener不是处理InvocationBusinessMethodFinishEvent事件而是InvocationBusinessFinishEvent事件。

我和宝哥说了下，他说可以优化一下。

[wujimin]

InvocationBusinessMethodFinishEvent仅仅表示controller方法执行完成
InvocationBusinessFinishEvent表示controller逻辑完成

对于同步：方法完成，就是逻辑完成
对于异步：方法完成，不等于逻辑完成，且方法完成是很快的（慢就是bug）

所以，统一在InvocationBusinessFinishEvent处理

[liubao68]

close old issues, please feel free to submit a new one if the problem still exists.