Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support async execute api. #366

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

heihei180
Copy link
Contributor

fixs: #365
support async execute api.

support async execute api.
@heihei180
Copy link
Contributor Author

heihei180 commented Nov 9, 2022

It seems that the unit test failed to connect to the database?

@oldratlee oldratlee force-pushed the master branch 4 times, most recently from 9f7e07b to b8eab09 Compare April 4, 2023 17:38
@oldratlee
Copy link
Member

@heihei180 please rebase master branch(and then force push to your branch)

Ivan97 and others added 2 commits April 7, 2023 11:23
1. 增加com.alibaba.cola.statemachine.StateMachine#verify方法
用来判断当前状态能否响应某个事件

2. 增加com.alibaba.cola.statemachine.builder.StateMachineBuilder#setFailoverCallback
用来增加在状态流转没有命中当前配置的规则时执行的回调
接口定义 com.alibaba.cola.statemachine.builder.FailoverCallback
默认为NumbFailoverCallbackImpl,与当前行为一样,什么都不做。
另外内置AlertFailoverCallbackImpl,用于抛出异常TransitionFailoverException
@heihei180
Copy link
Contributor Author

@heihei180 please rebase master branch(and then force push to your branch)

done

@oldratlee
Copy link
Member

@significantfrank this PR seems a good idea, how about reviewing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants