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

Jump to bottom

feature: StateMachine ServiceTask supports asynchronous execution #1844

Merged
merged 57 commits into from Nov 1, 2019
Merged

feature: StateMachine ServiceTask supports asynchronous execution #1844

merged 57 commits into from Nov 1, 2019

Conversation

long187
Copy link
Contributor

@long187 long187 commented Oct 30, 2019

Ⅰ. Describe what this PR did

Saga: StateMachine ServiceTask supports asynchronous execution

Ⅱ. Does this pull request fix one issue?

#1843

lorne.cl and others added 30 commits September 3, 2019 16:17
saga implement
d3c4cbe
saga implement
982cbd7
implement getStateMachineInstanceByBusinessKey
e2fe7e7
support table prefix
b7c891e
support transactions externally managed
8085000
state log store support transaction PropagationBehavior and Isolation…
2c4e3e0 
…Level
state log store support transaction PropagationBehavior and Isolation…
fd64918 
…Level
support mult tenants
e4c39ad
statelang json Status node expression as key, status as values
2a1964d
StateLogStore.getStateMachineInstanceByBusinessKey add parameter tena…
ad5bf43 
…ntId
schema sql script to lowercase
651ceca
change state type name "CompensationStart" to "CompensationTrigger"
b4d4966
improve DirectEventBus avoid stackoverflow
bf9d2f5
merge from upstream/develop
7638b6d
fix merge conflict
b5296f6
@long187
Merge branch 'develop' into develop
bffd21c
@long187
Merge branch 'develop' into develop
c9491de
remove mybatis,gson,commons dependencies
8a10c17
Merge branch 'develop' of https://github.com/long187/seata into develop
6c7bea4
merge from upstream
218b616
update saga test to junit5 and move to test project
597ec8a
Merge remote-tracking branch 'upstream/develop' into develop
a28bad9
@slievrly
Merge branch 'develop' into develop
7dd0791
support db2/oracle create schema scripts
a9ee505
Merge branch 'develop' of https://github.com/long187/seata into develop
a6fab5c
Merge remote-tracking branch 'upstream/develop' into develop
e3ed43b
fix mysql create table scripts
0a81952
Merge remote-tracking branch 'upstream/develop' into develop
5359e20
@long187
Merge branch 'develop' into develop
9bde6f6
fix code review optimization suggestions
fe1d071
lorne.cl and others added 22 commits September 26, 2019 15:56
Merge branch 'develop' of https://github.com/long187/seata into develop
bdf421c
fix tests and store status to db when record stateMachine started.
17c8409
add Feature.ordedFiled to JSONParser
b657ba4
Merge remote-tracking branch 'upstream/develop' into develop
f924921
refactor DbAndReportTcStateLogStore.java
f4cc7eb
Merge remote-tracking branch 'upstream/develop' into develop
cc94376
get dbType from datasoruce; if statemechine inst not exists, stop retry
423cd77
Merge remote-tracking branch 'upstream/develop' into develop
740abba
@l81893521
Merge branch 'develop' into develop
3ac4548
improve SpringBeanServiceInvoker.toJavaObject method, skip serialize …
129141e 
…when paramType isAssignableFrom value.getClass()
Merge branch 'develop' of https://github.com/long187/seata into develop
0b8408c
improve SpringBeanServiceInvoker.toJavaObject method, skip serialize …
549d85d 
…when paramType isAssignableFrom value.getClass()
merge from upstream
9cefd2d
merge from upstream
cd1af4c
Merge remote-tracking branch 'upstream/develop' into develop
806a1fc
@long187
Merge branch 'develop' into develop
a66721e
@slievrly
Merge branch 'develop' into develop
280bec8
Merge remote-tracking branch 'upstream/develop' into develop
6720c2d
release 0.9.0
c558cc8
@long187
merge from upstream
99855ba
@long187
Merge remote-tracking branch 'upstream/develop' into develop_#1829
23f00f2
@long187
Saga: StateMachine ServiceTask supports asynchronous execution apache…
d30ee80 
…#1843
@codecov-io
Copy link

codecov-io commented Oct 30, 2019 •
edited

Codecov Report

Merging #1844 into develop will decrease coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #1844      +/-   ##
=============================================
- Coverage      55.05%   55.05%   -0.01%     
- Complexity      2353     2356       +3     
=============================================
  Files            422      422              
  Lines          14271    14277       +6     
  Branches        1712     1713       +1     
=============================================
+ Hits            7857     7860       +3     
- Misses          5684     5687       +3     
  Partials         730      730
Impacted Files Coverage Δ Complexity Δ
.../statelang/parser/impl/ServiceTaskStateParser.java 83.33% <33.33%> (-16.67%) 2 <0> (ø)
...ga/statelang/domain/impl/ServiceTaskStateImpl.java 66.66% <33.33%> (-3.71%) 14 <1> (+1)
...server/store/file/FileTransactionStoreManager.java 59.86% <0%> (-0.33%) 29% <0%> (ø)
...o/seata/server/coordinator/DefaultCoordinator.java 50% <0%> (+0.4%) 28% <0%> (+1%) ⬆️
...in/java/io/seata/server/session/GlobalSession.java 85.02% <0%> (+0.48%) 68% <0%> (+1%) ⬆️

@slievrly slievrly changed the title Saga: StateMachine ServiceTask supports asynchronous execution feature: StateMachine ServiceTask supports asynchronous execution Oct 31, 2019
slievrly
slievrly approved these changes Oct 31, 2019
View reviewed changes
Copy link
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

zhangthen
zhangthen approved these changes Nov 1, 2019
View reviewed changes
Copy link
Contributor

@zhangthen zhangthen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@zhangthen zhangthen merged commit 29f805a into apache:develop Nov 1, 2019
@slievrly slievrly added this to the 1.0 milestone Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slievrly slievrly slievrly approved these changes

@zhangthen zhangthen zhangthen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@long187 @codecov-io @slievrly @zhangthen @l81893521