You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datalab.apache.org by yk...@apache.org on 2021/07/01 09:31:42 UTC

[incubator-datalab] branch develop updated: [DATALAB] -- fixed name for DP on AWS

This is an automated email from the ASF dual-hosted git repository.

ykinash pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/incubator-datalab.git


The following commit(s) were added to refs/heads/develop by this push:
     new bbea604  [DATALAB] -- fixed name for DP on AWS
     new 0ef7cfc  Merge remote-tracking branch 'origin/develop' into develop
bbea604 is described below

commit bbea6049923e712fe24df30ec925661e22a15f18
Author: KinashYurii <ur...@gmail.com>
AuthorDate: Thu Jul 1 12:30:02 2021 +0300

    [DATALAB] -- fixed name for DP on AWS
---
 .../aws/computational/ComputationalCreateAws.java  | 61 ++++------------------
 .../backendapi/resources/ExploratoryResource.java  |  2 +-
 .../dto/aws/AwsComputationalCreateForm.java        |  2 -
 3 files changed, 10 insertions(+), 55 deletions(-)

diff --git a/services/datalab-model/src/main/java/com/epam/datalab/dto/aws/computational/ComputationalCreateAws.java b/services/datalab-model/src/main/java/com/epam/datalab/dto/aws/computational/ComputationalCreateAws.java
index bb1eeb9..a2b044b 100644
--- a/services/datalab-model/src/main/java/com/epam/datalab/dto/aws/computational/ComputationalCreateAws.java
+++ b/services/datalab-model/src/main/java/com/epam/datalab/dto/aws/computational/ComputationalCreateAws.java
@@ -22,9 +22,13 @@ package com.epam.datalab.dto.aws.computational;
 import com.epam.datalab.dto.base.computational.ComputationalBase;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.google.common.base.MoreObjects.ToStringHelper;
+import lombok.Getter;
+import lombok.Setter;
 
 import java.util.List;
 
+@Getter
+@Setter
 public class ComputationalCreateAws extends ComputationalBase<ComputationalCreateAws> {
     @JsonProperty("emr_instance_count")
     private String instanceCount;
@@ -56,49 +60,21 @@ public class ComputationalCreateAws extends ComputationalBase<ComputationalCreat
         return this;
     }
 
-    public String getMasterInstanceType() {
-        return masterInstanceType;
-    }
-
-    public void setMasterInstanceType(String masterInstanceType) {
-        this.masterInstanceType = masterInstanceType;
-    }
-
     public ComputationalCreateAws withMasterInstanceType(String masterInstanceType) {
         setMasterInstanceType(masterInstanceType);
         return this;
     }
 
-    public String getSlaveInstanceType() {
-        return slaveInstanceType;
-    }
-
-    public void setSlaveInstanceType(String slaveInstanceType) {
-        this.slaveInstanceType = slaveInstanceType;
-    }
-
     public ComputationalCreateAws withSlaveInstanceType(String slaveInstanceType) {
         setSlaveInstanceType(slaveInstanceType);
         return this;
     }
 
-    public Boolean getSlaveInstanceSpot() {
-        return slaveInstanceSpot;
-    }
-
-    public void setSlaveInstanceSpot(Boolean slaveInstanceSpot) {
-        this.slaveInstanceSpot = slaveInstanceSpot;
-    }
-
     public ComputationalCreateAws withSlaveInstanceSpot(Boolean slaveInstanceSpot) {
         setSlaveInstanceSpot(slaveInstanceSpot);
         return this;
     }
 
-    public Integer getSlaveInstanceSpotPctPrice() {
-        return slaveInstanceSpotPctPrice;
-    }
-
     public void setSlaveInstanceSpotPctPrice(Integer slaveInstanceSpotPctPrice) {
         this.slaveInstanceSpotPctPrice = slaveInstanceSpotPctPrice;
     }
@@ -108,45 +84,26 @@ public class ComputationalCreateAws extends ComputationalBase<ComputationalCreat
         return this;
     }
 
-    public String getVersion() {
-        return version;
-    }
-
-    public void setVersion(String version) {
-        this.version = version;
-    }
-
     public ComputationalCreateAws withVersion(String version) {
         setVersion(version);
         return this;
     }
 
-    public List<ClusterConfig> getConfig() {
-        return config;
-    }
-
-    public void setConfig(List<ClusterConfig> config) {
-        this.config = config;
-    }
-
     public ComputationalCreateAws withConfig(List<ClusterConfig> config) {
         setConfig(config);
         return this;
     }
 
-    public String getSharedImageEnabled() {
-        return sharedImageEnabled;
-    }
-
-    public void setSharedImageEnabled(String sharedImageEnabled) {
-        this.sharedImageEnabled = sharedImageEnabled;
-    }
-
     public ComputationalCreateAws withSharedImageEnabled(String sharedImageEnabled) {
         setSharedImageEnabled(sharedImageEnabled);
         return this;
     }
 
+    public ComputationalCreateAws withNotebookName(String name) {
+        setNotebookInstanceName(name);
+        return this;
+    }
+
     @Override
     public ToStringHelper toStringHelper(Object self) {
         return super.toStringHelper(self)
diff --git a/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/ExploratoryResource.java b/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/ExploratoryResource.java
index 43dbc9c..8d5cfe0 100644
--- a/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/ExploratoryResource.java
+++ b/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/ExploratoryResource.java
@@ -155,7 +155,7 @@ public class ExploratoryResource implements ExploratoryAPI {
         return Exploratory.builder()
                 .name(formDTO.getName())
                 .dockerImage(formDTO.getImage())
-                .imageName(formDTO.getImageName())
+                .imageName(formDTO.getImageName().isEmpty() ? formDTO.getVersion() : formDTO.getImageName())
                 .templateName(formDTO.getTemplateName())
                 .version(formDTO.getVersion())
                 .clusterConfig(formDTO.getClusterConfig())
diff --git a/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/dto/aws/AwsComputationalCreateForm.java b/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/dto/aws/AwsComputationalCreateForm.java
index 5b536ee..314dba5 100644
--- a/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/dto/aws/AwsComputationalCreateForm.java
+++ b/services/self-service/src/main/java/com/epam/datalab/backendapi/resources/dto/aws/AwsComputationalCreateForm.java
@@ -52,6 +52,4 @@ public class AwsComputationalCreateForm extends ComputationalCreateFormDTO {
     @NotBlank
     @JsonProperty("emr_version")
     private String version;
-
-
 }

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datalab.apache.org
For additional commands, e-mail: commits-help@datalab.apache.org