Skip to content

Add Jobs quickstart #1198

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.

Sign up for GitHub

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
Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open

Add Jobs quickstart #1198

wants to merge 56 commits into from

Conversation

siri-varma
Copy link
Contributor

@siri-varma siri-varma commented May 27, 2025 •
edited
Loading

Description

Add Quick start guide for Jobs API

Issue reference

We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1115

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • The quickstart code compiles correctly
  • You've tested new builds of the quickstart if you changed quickstart code
  • You've updated the quickstart's README if necessary
  • If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

@siri-varma siri-varma requested review from a team as code owners May 27, 2025 22:34
@siri-varma siri-varma force-pushed the users/svegiraju/jobs-quickstart branch from 63973bb to 2316429 Compare May 28, 2025 05:34
@siri-varma
Copy link
Contributor Author

@dapr-java-sdk-reviewers @cicoyle @salaboy @artur-ciocanu This PR is also good to go. Just need approval for the workflows

@salaboy
Copy link
Contributor

salaboy commented May 28, 2025

@siri-varma I agree. From a (pure) Java point of view this is ready to be merged.

@salaboy
Copy link
Contributor

salaboy commented May 28, 2025

/LGTM

@alicejgibbons alicejgibbons added the do-not-merge Do not merge the PR label May 28, 2025
alicejgibbons
alicejgibbons requested changes May 28, 2025
View reviewed changes
Copy link
Contributor

@alicejgibbons alicejgibbons left a comment

Choose a reason for hiding this comment

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

Thanks @siri-varma, a couple changes for consistency

jobs/java/sdk/job-scheduler/pom.xml
<dependency>
<groupId>io.dapr</groupId>
<artifactId>dapr-sdk</artifactId>
<version>1.15.0-rc-5</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Will wait to merge until 1.15.0 is out

marcduiker and others added 22 commits May 28, 2025 13:25
@marcduiker @siri-varma
Add fundamentals example for Python
8bd32a4 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add comments and types
cde572a 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Change import, add task chaining and fan-in fan-out
aad6f0e 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Use lifespan for wf_runtime, add monitor
ff1003c 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Remove logging
413627d 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add external events, use flush true for print statements
3c18d45 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add child workflows
c396ef6 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add workflow management
e2592e4 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Revert dapr.yaml changes for C#
4f19e8f 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add resiliency and compensation
b175f0a 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add Python README
d12d612 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add combined patterns (WIP)
ece50b7 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Update orderworkflow
b2842f9 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Add challenges and tips code
4c71f90 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Try converting from SimpleNamespace to dataclass
19cb445 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Change to pydantic models
88748b9 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Change to model_dump
32bf96c 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @Dzvezdana @siri-varma
Update tutorials/workflow/python/monitor-pattern/monitor/monitor_work…
b106f26 
…flow.py

Co-authored-by: Dzvezdana Arsovska <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @Dzvezdana @siri-varma
Update tutorials/workflow/python/fan-out-fan-in/fan_out_fan_in/fanout…
3fb9bc3 
…fanin_workflow.py

Co-authored-by: Dzvezdana Arsovska <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @Dzvezdana @siri-varma
Update tutorials/workflow/python/monitor-pattern/monitor/monitor_work…
75a0442 
…flow.py

Co-authored-by: Dzvezdana Arsovska <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @Dzvezdana @siri-varma
Update tutorials/workflow/python/monitor-pattern/monitor/monitor_work…
3985325 
…flow.py

Co-authored-by: Dzvezdana Arsovska <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @Dzvezdana @siri-varma
Update tutorials/workflow/python/monitor-pattern/monitor/monitor_work…
16c62e2 
…flow.py

Co-authored-by: Dzvezdana Arsovska <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
marcduiker and others added 23 commits May 28, 2025 13:25
@marcduiker @siri-varma
Fix workflow serialization
f23b769 
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/fundamentals/basic/basic_workflow.py
aef9101 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/workflow-management/README.md
dfa70d9 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/workflow-management/README.md
decfde1 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/workflow-management/README.md
d70be34 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/task-chaining/README.md
15b4dc5 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/workflow-management/README.md
1dba48d 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/combined-patterns/dapr.yaml
b86bacc 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/combined-patterns/dapr.yaml
14702c2 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/workflow-management/README.md
785c595 
Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @alicejgibbons @siri-varma
Update tutorials/workflow/python/challenges-tips/payload_size_workflo…
fe1ba3c 
…w.py

Co-authored-by: Alice Gibbons <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@marcduiker @siri-varma
Update tutorials/workflow/python/monitor-pattern/monitor/monitor_work…
2b2c899 
…flow.py

Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@tommygood @siri-varma
fix(order-processor): provide default DAPR_HTTP_PORT fallback (3500) …
e92749b 
…in state_management/javascript.

Signed-off-by: tommygood <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Add Jobs quickstart
e5a77e1 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
REmove spaces (dapr#1197)
c21ae11 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Modify yaml
2d004a4 
Signed-off-by: Siri Varma Vegiraju <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Amend
083db68 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Amend
9b69dd1 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Fix yaml
4c808bf 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Fix job
bb86a28 
Signed-off-by: Siri Varma Vegiraju <[email protected]>
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Play
ef4da09 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Fix things
c59ef9c 
Signed-off-by: siri-varma <[email protected]>
@siri-varma
Fix readme
d242992 
Signed-off-by: siri-varma <[email protected]>
@siri-varma siri-varma force-pushed the users/svegiraju/jobs-quickstart branch from bddf466 to d242992 Compare May 28, 2025 20:25
@siri-varma
Copy link
Contributor Author

@alicejgibbons Added the api to receive the triggers as well.

@alicejgibbons
Copy link
Contributor

@alicejgibbons Added the api to receive the triggers as well.

Thanks @siri-varma looks good now. In my comments I had actually meant that we still could have used one service but had the client and the handers in the same app as per Whit's example here: https://github.com/dapr/quickstarts/blob/d1a7f7ca59291a5e744bf9a03b4db4588100d790/jobs/csharp/http/job-service/Program.cs

Anyways, the more i think about it the more im not sure i love the one app solution better since it might be more likely that users will split out the job invocation in actual scenarios. So all this to say is this can be merged once we get 1.15 :)

@siri-varma
Copy link
Contributor Author

@alicejgibbons small request, could you please trigger the build for this PR ? Want to make sure it passes so that we will not have any suprises at the end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alicejgibbons alicejgibbons alicejgibbons requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
do-not-merge Do not merge the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

SDK Java Jobs API Quickstart
5 participants
@siri-varma @salaboy @alicejgibbons @marcduiker @tommygood