Update SDK POJOs to have Lombok annotations (#463)

moves all java sdk POJOs to lombok annotations.
We will have
@DaTa
@builder
@NoArgsConstructor
@AllArgsConstructor

Our goal is to not break current backward compatibility. We have added unit tests in a previous PR for all the accessors and constructors which are deleted as part of Lombokisation.

This will be a base on which we will sync sdk POJOs with server POJOs.

Author: orkes-harshil

conductor-clients/java/conductor-java-sdk/conductor-client/src/main/java/com/netflix/conductor/common/metadata/Auditable.java

@@ -12,6 +12,14 @@
  */
 package com.netflix.conductor.common.metadata;
 
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Data;
+import lombok.NoArgsConstructor;
+
+@Data
+@NoArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public abstract class Auditable {
 
     private String ownerApp;
@@ -24,73 +32,11 @@ public abstract class Auditable {
 
     private String updatedBy;
 
-    /**
-     * @return the ownerApp
-     */
-    public String getOwnerApp() {
-        return ownerApp;
-    }
-
-    /**
-     * @param ownerApp the ownerApp to set
-     */
-    public void setOwnerApp(String ownerApp) {
-        this.ownerApp = ownerApp;
-    }
-
-    /**
-     * @return the createTime
-     */
     public Long getCreateTime() {
         return createTime == null ? 0 : createTime;
     }
 
-    /**
-     * @param createTime the createTime to set
-     */
-    public void setCreateTime(Long createTime) {
-        this.createTime = createTime;
-    }
-
-    /**
-     * @return the updateTime
-     */
     public Long getUpdateTime() {
         return updateTime == null ? 0 : updateTime;
     }
-
-    /**
-     * @param updateTime the updateTime to set
-     */
-    public void setUpdateTime(Long updateTime) {
-        this.updateTime = updateTime;
-    }
-
-    /**
-     * @return the createdBy
-     */
-    public String getCreatedBy() {
-        return createdBy;
-    }
-
-    /**
-     * @param createdBy the createdBy to set
-     */
-    public void setCreatedBy(String createdBy) {
-        this.createdBy = createdBy;
-    }
-
-    /**
-     * @return the updatedBy
-     */
-    public String getUpdatedBy() {
-        return updatedBy;
-    }
-
-    /**
-     * @param updatedBy the updatedBy to set
-     */
-    public void setUpdatedBy(String updatedBy) {
-        this.updatedBy = updatedBy;
-    }
-}
+}

conductor-clients/java/conductor-java-sdk/conductor-client/src/main/java/com/netflix/conductor/common/metadata/SchemaDef.java

@@ -40,4 +40,5 @@ public enum Type {
     // Externalized schema definition (eg. via AVRO, Protobuf registry)
     // If using Orkes Schema registry, this points to the name of the schema in the registry
     private String externalRef;
+
 }

conductor-clients/java/conductor-java-sdk/conductor-client/src/main/java/com/netflix/conductor/common/metadata/events/EventExecution.java

@@ -18,6 +18,12 @@
 
 import com.netflix.conductor.common.metadata.events.EventHandler.Action;
 
+import lombok.*;
+
+@Data
+@NoArgsConstructor
+@Builder
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class EventExecution {
 
     public enum Status {
@@ -41,126 +47,11 @@ public enum Status {
 
     private Map<String, Object> output = new HashMap<>();
 
-    public EventExecution() {
-    }
-
     public EventExecution(String id, String messageId) {
         this.id = id;
         this.messageId = messageId;
     }
 
-    /**
-     * @return the id
-     */
-    public String getId() {
-        return id;
-    }
-
-    /**
-     * @param id the id to set
-     */
-    public void setId(String id) {
-        this.id = id;
-    }
-
-    /**
-     * @return the messageId
-     */
-    public String getMessageId() {
-        return messageId;
-    }
-
-    /**
-     * @param messageId the messageId to set
-     */
-    public void setMessageId(String messageId) {
-        this.messageId = messageId;
-    }
-
-    /**
-     * @return the name
-     */
-    public String getName() {
-        return name;
-    }
-
-    /**
-     * @param name the name to set
-     */
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    /**
-     * @return the event
-     */
-    public String getEvent() {
-        return event;
-    }
-
-    /**
-     * @param event the event to set
-     */
-    public void setEvent(String event) {
-        this.event = event;
-    }
-
-    /**
-     * @return the created
-     */
-    public long getCreated() {
-        return created;
-    }
-
-    /**
-     * @param created the created to set
-     */
-    public void setCreated(long created) {
-        this.created = created;
-    }
-
-    /**
-     * @return the status
-     */
-    public Status getStatus() {
-        return status;
-    }
-
-    /**
-     * @param status the status to set
-     */
-    public void setStatus(Status status) {
-        this.status = status;
-    }
-
-    /**
-     * @return the action
-     */
-    public Action.Type getAction() {
-        return action;
-    }
-
-    /**
-     * @param action the action to set
-     */
-    public void setAction(Action.Type action) {
-        this.action = action;
-    }
-
-    /**
-     * @return the output
-     */
-    public Map<String, Object> getOutput() {
-        return output;
-    }
-
-    /**
-     * @param output the output to set
-     */
-    public void setOutput(Map<String, Object> output) {
-        this.output = output;
-    }
-
     public boolean equals(Object o) {
         if (this == o) {
             return true;
@@ -175,4 +66,4 @@ public boolean equals(Object o) {
     public int hashCode() {
         return Objects.hash(id, messageId, name, event, created, status, action, output);
     }
-}
+}

conductor-clients/java/conductor-java-sdk/conductor-client/src/main/java/com/netflix/conductor/common/metadata/events/EventHandler.java

@@ -17,11 +17,21 @@
 import java.util.List;
 import java.util.Map;
 
+import lombok.*;
+
 /**
  * Defines an event handler
  */
+
+@Data
+@NoArgsConstructor
+@Builder
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class EventHandler {
 
+    /**
+     * the name MUST be unique within a conductor instance
+     */
     private String name;
 
     private String event;
@@ -34,93 +44,6 @@ public class EventHandler {
 
     private String evaluatorType;
 
-    public EventHandler() {
-    }
-
-    /**
-     * @return the name MUST be unique within a conductor instance
-     */
-    public String getName() {
-        return name;
-    }
-
-    /**
-     * @param name the name to set
-     */
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    /**
-     * @return the event
-     */
-    public String getEvent() {
-        return event;
-    }
-
-    /**
-     * @param event the event to set
-     */
-    public void setEvent(String event) {
-        this.event = event;
-    }
-
-    /**
-     * @return the condition
-     */
-    public String getCondition() {
-        return condition;
-    }
-
-    /**
-     * @param condition the condition to set
-     */
-    public void setCondition(String condition) {
-        this.condition = condition;
-    }
-
-    /**
-     * @return the actions
-     */
-    public List<Action> getActions() {
-        return actions;
-    }
-
-    /**
-     * @param actions the actions to set
-     */
-    public void setActions(List<Action> actions) {
-        this.actions = actions;
-    }
-
-    /**
-     * @return the active
-     */
-    public boolean isActive() {
-        return active;
-    }
-
-    /**
-     * @param active if set to false, the event handler is deactivated
-     */
-    public void setActive(boolean active) {
-        this.active = active;
-    }
-
-    /**
-     * @return the evaluator type
-     */
-    public String getEvaluatorType() {
-        return evaluatorType;
-    }
-
-    /**
-     * @param evaluatorType the evaluatorType to set
-     */
-    public void setEvaluatorType(String evaluatorType) {
-        this.evaluatorType = evaluatorType;
-    }
-
     public static class Action {
 
         public enum Type {