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

fix(AAE-201): Fixed the problem when process deployed and start message events are dispatched twice #2986

Merged
merged 1 commit into from
Nov 19, 2019

Conversation

igdianov
Copy link
Contributor

This PR fixes problem when the process deployed and start message events are dispatched twice using Spring ApplicationReady event

Part of #1691

in order to avoid duplicate process deployed and start message deployed events
@codecov
Copy link

codecov bot commented Nov 16, 2019

Codecov Report

Merging #2986 into develop will increase coverage by 0.02%.
The diff coverage is 84.9%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2986      +/-   ##
=============================================
+ Coverage      61.91%   61.94%   +0.02%     
  Complexity      1607     1607              
=============================================
  Files           1202     1203       +1     
  Lines          41861    41900      +39     
  Branches        5986     5988       +2     
=============================================
+ Hits           25918    25954      +36     
  Misses         13137    13137              
- Partials        2806     2809       +3
Impacted Files Coverage Δ Complexity Δ
...viti/spring/StartMessageDeployedEventProducer.java 95.74% <100%> (+0.18%) 0 <0> (ø) ⬇️
.../activiti/spring/ProcessDeployedEventProducer.java 87.5% <78.57%> (ø) 0 <0> (ø) ⬇️
...ctiviti/spring/AbstractActivitiSmartLifeCycle.java 86.48% <86.48%> (ø) 0 <0> (?)
...mpl/asyncexecutor/AcquireAsyncJobsDueRunnable.java 71.92% <0%> (-3.51%) 0% <0%> (ø)
...mpl/agenda/TakeOutgoingSequenceFlowsOperation.java 89.8% <0%> (+1.27%) 0% <0%> (ø) ⬇️
...e/impl/asyncexecutor/ResetExpiredJobsRunnable.java 68.29% <0%> (+4.87%) 0% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 509b014...605c4c4. Read the comment docs.

@mergify mergify bot merged commit f6eaf94 into develop Nov 19, 2019
@mergify mergify bot deleted the fix-AAE-201-duplicate-events branch November 19, 2019 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants