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

What is the jobId in the triggerWorkflow result? #79

Open
chrisli30 opened this issue Dec 16, 2024 · 4 comments
Open

What is the jobId in the triggerWorkflow result? #79

chrisli30 opened this issue Dec 16, 2024 · 4 comments
Labels
question Further information is requested

Comments

@chrisli30
Copy link
Member

What is the jobId in this response of triggerWorkflow? Could you write the explanation here so I can include it in the sdk?

It’s generated from one of the triggerWorkflow test cases.
AvaProtocol/ava-sdk-js#45

{ result: true, executionId: '', jobId: '4' }
@chrisli30 chrisli30 added the question Further information is requested label Dec 16, 2024
@v9n
Copy link
Member

v9n commented Dec 16, 2024

@chrisli30 > Could you write the explanation here so I can include it in the sdk?

It's an internal job of that execution in the worker queue. Right now, client cannot do anything with it yet but maybe in the furure.

@chrisli30
Copy link
Member Author

What’s the relationship between the executionId and the jobId?

Is jobId sub-tasks of an execution, and starts from 0?

@v9n
Copy link
Member

v9n commented Dec 17, 2024

@chrisli30

What’s the relationship between the executionId and the jobId?

there is no direct relationship currently.

Is jobId sub-tasks of an execution, and starts from 0?

no, it isn't

It is just simply an ID of an internal queue system that used to run these job. When a job is run async, they are enqueue to be run. That's the internal id in that queue. A worker pop the work from that queue to run.

I'm going to remove this field to avid any confusion about its usage or id. So when run a job is trigger in async manner. Sound good to you?

@chrisli30
Copy link
Member Author

okay, so sounds like it’s for internal use only? If that’s the case, let’s remove it from the API response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants