Skip to content

Add waitForStateExecutionCompletion API #207

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

Merged
merged 7 commits into from
Oct 28, 2023
Merged

Add waitForStateExecutionCompletion API #207

merged 7 commits into from
Oct 28, 2023

Conversation

longquanzheng
Copy link
Contributor

No description provided.

@duoertai
Copy link
Collaborator

RpcTest > testRPCLocking() FAILED
feign.RetryableException at RpcTest.java:93
Caused by: feign.FeignException$InternalServerError at RpcTest.java:93

@duoertai
Copy link
Collaborator

[500 Internal Server Error] during [POST] to [http://localhost:8801/api/v1/workflow/rpc] [DefaultApi#apiV1WorkflowRpcPost(WorkflowRpcRequest)]: [{"detail":"Workflow task is not scheduled yet.","subStatus":"UNCATEGORIZED_SUB_STATUS"}]

@duoertai
Copy link
Collaborator

duoertai commented Sep 26, 2023

ran the test at my local, it seems when the test call invoke rpc -> getCounter, the workflow is stuck for a while with the 1st workflow task not scheduled,

Screenshot 2023-09-26 at 12 12 47 AM

@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #207 (d4c6f9e) into main (d2ae843) will increase coverage by 0.35%.
The diff coverage is 69.23%.

@@             Coverage Diff              @@
##               main     #207      +/-   ##
============================================
+ Coverage     71.93%   72.29%   +0.35%     
- Complexity      389      392       +3     
============================================
  Files            61       61              
  Lines          1593     1606      +13     
  Branches        140      141       +1     
============================================
+ Hits           1146     1161      +15     
+ Misses          376      373       -3     
- Partials         71       72       +1     
Files Coverage Δ
src/main/java/io/iworkflow/core/Client.java 63.02% <100.00%> (+0.31%) ⬆️
...io/iworkflow/core/UnregisteredWorkflowOptions.java 100.00% <ø> (ø)
...c/main/java/io/iworkflow/core/WorkflowOptions.java 100.00% <ø> (ø)
...ain/java/io/iworkflow/core/UnregisteredClient.java 62.01% <63.63%> (+0.09%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@longquanzheng longquanzheng merged commit b87c388 into main Oct 28, 2023
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.

2 participants