conductor icon indicating copy to clipboard operation
conductor copied to clipboard

Published 20 hours ago • verified publisher icon Netflix

Update WorkflowExecutor.java

Open youngledo opened this issue 8 months ago • 2 comments

Pull Request type

  • [ ] Bugfix
  • [x] Feature
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • [ ] WHOSUSING.md
  • [ ] Other (please describe):

Changes in this PR

The business requires "workflowId" to obtain the runtime process.

youngledo avatar Dec 01 '23 07:12 youngledo

Hi @youngledo , Workflow already contains workflowId. Is there a reason to change the behaviour to just return workflowId and not workflow model?

manan164 avatar Dec 04 '23 13:12 manan164

Hi @youngledo , Workflow already contains workflowId. Is there a reason to change the behaviour to just return workflowId and not workflow model?

Sorry, I didn't make it clear.

image The "workflowId" in this method is only cached in "runningWorkflowFutures" and cannot be obtained by external sources. image The workflow model object results are returned in the scheduling thread pool and cannot be obtained in real time.

youngledo avatar Dec 05 '23 02:12 youngledo