small fix

Author: guoqiao1992

src/sagemaker/apiutils/_base_types.py

@@ -42,21 +42,23 @@ def _boto_ignore(cls):
         return ["ResponseMetadata"]
 
     @classmethod
-    def from_boto(cls, boto_dict, **kwargs):  # pylint: disable=R1710
+    def from_boto(cls, boto_dict, **kwargs):
         """Construct an instance of this ApiObject from a boto response.
 
         Args:
             boto_dict (dict): A dictionary of a boto response.
             **kwargs: Arbitrary keyword arguments
         """
-        if boto_dict:
-            boto_dict = {k: v for k, v in boto_dict.items() if k not in cls._boto_ignore()}
-            custom_boto_names_to_member_names = {a: b for b, a in cls._custom_boto_names.items()}
-            cls_kwargs = _boto_functions.from_boto(
-                boto_dict, custom_boto_names_to_member_names, cls._custom_boto_types
-            )
-            cls_kwargs.update(kwargs)
-            return cls(**cls_kwargs)
+        if not boto_dict:
+            return None
+
+        boto_dict = {k: v for k, v in boto_dict.items() if k not in cls._boto_ignore()}
+        custom_boto_names_to_member_names = {a: b for b, a in cls._custom_boto_names.items()}
+        cls_kwargs = _boto_functions.from_boto(
+            boto_dict, custom_boto_names_to_member_names, cls._custom_boto_types
+        )
+        cls_kwargs.update(kwargs)
+        return cls(**cls_kwargs)
 
     @classmethod
     def to_boto(cls, obj):

src/sagemaker/processing.py

@@ -1024,8 +1024,6 @@ def __init__(
     def _to_request_dict(self):
         """Generates a request dictionary using the parameters provided to the class."""
 
-        # self._create_s3_input()
-
         # Create the request dictionary.
         s3_input_request = {"InputName": self.input_name, "AppManaged": self.app_managed}
 
@@ -1062,9 +1060,7 @@ def _create_s3_input(self):
             self.s3_data_type = self.s3_input.s3_data_type
             self.s3_input_mode = self.s3_input.s3_input_mode
             self.s3_data_distribution_type = self.s3_input.s3_data_distribution_type
-            return
-
-        if self.source and self.destination:
+        elif self.source and self.destination:
             self.s3_input = S3Input(
                 s3_uri=self.source,
                 local_path=self.destination,