Merge pull request #13 from AlexanderMann/performance/remove-command-is-redundant

Performance: REMOVEing from an internal table stage before a DROP TABLE command is redundant

Author: AlexanderMann

target_snowflake/snowflake.py

@@ -413,14 +413,14 @@ def perform_update(self, cur, target_table_name, temp_table_name, key_properties
                 insert_columns=insert_columns,
                 dedupped_columns=dedupped_columns))
 
-        # Clear out the associated stage for the table
-        cur.execute('''
-            REMOVE @{db}.{schema}.%{temp_table}
-        '''.format(
-            db=sql.identifier(self.connection.configured_database),
-            schema=sql.identifier(self.connection.configured_schema),
-            temp_table=sql.identifier(temp_table_name)
-        ))
+        if not self.s3:
+            # Clear out the associated stage for the table
+            cur.execute('''
+                REMOVE @{db}.{schema}.%{temp_table}
+            '''.format(
+                db=sql.identifier(self.connection.configured_database),
+                schema=sql.identifier(self.connection.configured_schema),
+                temp_table=sql.identifier(temp_table_name)))
 
         # Drop the tmp table
         cur.execute('''