-
Notifications
You must be signed in to change notification settings - Fork 212
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
Add Jobs API #1101
Comments
@cicoyle Can I work on this? |
@charan2628 Absolutely! Please assign yourself this issue by commenting ‘/assign’. Definitely refer to the go sdk example and reach out to me for additional assistance as needed |
/assign |
@cicoyle Can you please verify once my changes, so I can continue this approach |
@charan2628 we will look into your changes.. I think that @artursouza already made some comments. |
I believe there is still pending work here. I would like to take it up. Let me know if that's ok. |
@siri-varma - All community contributions are welcome :) I can assign you to this issue as well. @charan2628 has started on this work, but if you would like to pick up where it was left off we appreciate the help! Since the PR has not been updated since October that is fine by me if you would like to start a new branch, or if you'd like to cherry pick in what has already been started on. We will be sure to reference @charan2628 as they definitely put in some efforts. It will be appreciated if you can bring it over the line @siri-varma - we can get it in the next release |
@cicoyle I didn't received feedback for my changes, that's why I stopped working on this. I am asking in my PR if my implementation is fine, got no reply. |
@charan2628 there is an open PR showing how to add a new API to the Dapr SDK, which is quite complete: #1235 my advice would be to copy how that PR is adding the APIs and all the tests that needs to be added to check that the API is working correctly. Apologies for the delay in getting back to you with this PR. |
Ok @salaboy |
@siri-varma are you going to completely work on this |
@charan2628. Yes, I will be able to wrap this up. |
Dapr v1.14 adds support for the Jobs API which uses the Scheduler control plane service under the hood. This new API allows users to schedule/get/delete jobs to be triggered at some point in the future. Once triggered from the Scheduler control plane service, daprd will send the job to the app via either grpc or http via OnJobEventAlpha1 or /jobs repspectively. The API is exposed to the user for both grpc + http via the ScheduleJobAlpha1/GetJobAlpha1/DeleteJobAlpha1 apis.
Reference implementation for the go-sdk.
Note the API is called
jobs
The text was updated successfully, but these errors were encountered: