Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Commit

Permalink
Merge pull request #1864 from jinwik/bug-fix
Browse files Browse the repository at this point in the history
fix priority bug
  • Loading branch information
apanicker-nflx committed Sep 14, 2020
2 parents db2d1cb + 5c0b41b commit 749f922
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
Expand Up @@ -50,7 +50,7 @@ public class StartWorkflowRequest {
@ProtoField(id = 8)
@Min(value = 0, message = "priority: ${validatedValue} should be minimum {value}")
@Max(value = 99, message = "priority: ${validatedValue} should be maximum {value}")
private Integer priority;
private Integer priority = 0;

public String getName() {
return name;
Expand Down
Expand Up @@ -73,8 +73,9 @@ public WorkflowServiceImpl(WorkflowExecutor workflowExecutor, ExecutionService e
*/
@Service
public String startWorkflow(StartWorkflowRequest startWorkflowRequest) {
return startWorkflow(startWorkflowRequest.getName(), startWorkflowRequest.getVersion(), startWorkflowRequest.getCorrelationId(), startWorkflowRequest.getInput(),
startWorkflowRequest.getExternalInputPayloadStoragePath(), startWorkflowRequest.getTaskToDomain(), startWorkflowRequest.getWorkflowDef());
return startWorkflow(startWorkflowRequest.getName(), startWorkflowRequest.getVersion(), startWorkflowRequest.getCorrelationId(),
startWorkflowRequest.getPriority(), startWorkflowRequest.getInput(), startWorkflowRequest.getExternalInputPayloadStoragePath(),
startWorkflowRequest.getTaskToDomain(), startWorkflowRequest.getWorkflowDef());
}

/**
Expand Down
Expand Up @@ -62,6 +62,7 @@ public void startWorkflow(StartWorkflowRequestPb.StartWorkflowRequest pbRequest,
try {
String id = workflowService.startWorkflow(pbRequest.getName(),
GRPC_HELPER.optional(request.getVersion()),request.getCorrelationId(),
request.getPriority(),
request.getInput(),
request.getExternalInputPayloadStoragePath(),
request.getTaskToDomain(), request.getWorkflowDef());
Expand Down

0 comments on commit 749f922

Please sign in to comment.