Skip to content

Commit

Permalink
review comments.
Browse files Browse the repository at this point in the history
  • Loading branch information
manan164 committed Nov 29, 2023
1 parent 68c8105 commit 1a17091
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
import javax.validation.constraints.Max;
import javax.validation.constraints.Min;

import com.netflix.conductor.common.metadata.workflow.IdempotencyStrategy;
import org.apache.commons.lang3.StringUtils;

import com.netflix.conductor.annotations.protogen.ProtoEnum;
Expand Down Expand Up @@ -127,12 +126,8 @@ public boolean isSuccessful() {
@ProtoField(id = 25)
private List<Workflow> history = new LinkedList<>();

@ProtoField(id = 26)
private String idempotencyKey;

@ProtoField(id = 27)
private String idempotencyStrategy;

public String getIdempotencyKey() {
return idempotencyKey;
}
Expand All @@ -141,14 +136,6 @@ public void setIdempotencyKey(String idempotencyKey) {
this.idempotencyKey = idempotencyKey;
}

public String getIdempotencyStrategy() {
return idempotencyStrategy;
}

public void setIdempotencyStrategy(String idempotencyStrategy) {
this.idempotencyStrategy = idempotencyStrategy;
}

public List<Workflow> getHistory() {
return history;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1082,9 +1082,6 @@ public WorkflowPb.Workflow toProto(Workflow from) {
for (Workflow elem : from.getHistory()) {
to.addHistory( toProto(elem) );
}
if (from.getIdempotencyKey() != null) {
to.setIdempotencyKey( from.getIdempotencyKey() );
}
return to.build();
}

Expand Down Expand Up @@ -1125,7 +1122,6 @@ public Workflow fromProto(WorkflowPb.Workflow from) {
to.setVariables(variablesMap);
to.setLastRetriedTime( from.getLastRetriedTime() );
to.setHistory( from.getHistoryList().stream().map(this::fromProto).collect(Collectors.toCollection(ArrayList::new)) );
to.setIdempotencyKey( from.getIdempotencyKey() );
return to;
}

Expand Down

0 comments on commit 1a17091

Please sign in to comment.