🎉 Destination Snowflake: tag snowflake traffic with airbyte ID to ena…

…ble optimizations from Snowflake (#4713)
airbytehq · Jul 13, 2021 · 258a905 · 258a905
1 parent 3add914
commit 258a905
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 10 deletions.
diff --git a/...esources/config/STANDARD_DESTINATION_DEFINITION/424892c4-daac-4491-b35d-c6688ba547ba.json b/...esources/config/STANDARD_DESTINATION_DEFINITION/424892c4-daac-4491-b35d-c6688ba547ba.json
@@ -2,6 +2,6 @@
   "destinationDefinitionId": "424892c4-daac-4491-b35d-c6688ba547ba",
   "name": "Snowflake",
   "dockerRepository": "airbyte/destination-snowflake",
-  "dockerImageTag": "0.3.9",
+  "dockerImageTag": "0.3.10",
   "documentationUrl": "https://docs.airbyte.io/integrations/destinations/snowflake"
 }
diff --git a/airbyte-config/init/src/main/resources/seed/destination_definitions.yaml b/airbyte-config/init/src/main/resources/seed/destination_definitions.yaml
@@ -32,7 +32,7 @@
 - destinationDefinitionId: 424892c4-daac-4491-b35d-c6688ba547ba
   name: Snowflake
   dockerRepository: airbyte/destination-snowflake
-  dockerImageTag: 0.3.9
+  dockerImageTag: 0.3.10
   documentationUrl: https://docs.airbyte.io/integrations/destinations/snowflake
 - destinationDefinitionId: 4816b78f-1489-44c1-9060-4b19d5fa9362
   name: S3

diff --git a/...main/java/io/airbyte/integrations/standardtest/destination/DestinationAcceptanceTest.java b/...main/java/io/airbyte/integrations/standardtest/destination/DestinationAcceptanceTest.java
@@ -372,8 +372,10 @@ public void testSecondSync() throws Exception {
                     .put("id", 1)
                     .put("currency", "USD")
                     .put("date", "2020-03-31T00:00:00Z")
-                    .put("HKD", 10.0)
-                    .put("NZD", 700.0)
+                    // TODO(sherifnada) hack: write decimals with sigfigs because Snowflake stores 10.1 as "10" which
+                    // fails destination tests
+                    .put("HKD", 10.1)
+                    .put("NZD", 700.1)
                     .build()))),
         new AirbyteMessage()
             .withType(Type.STATE)
@@ -405,8 +407,10 @@ public void testLineBreakCharacters() throws Exception {
                     .put("id", 1)
                     .put("currency", "USD\u2028")
                     .put("date", "2020-03-\n31T00:00:00Z\r")
-                    .put("HKD", 10.0)
-                    .put("NZD", 700.0)
+                    // TODO(sherifnada) hack: write decimals with sigfigs because Snowflake stores 10.1 as "10" which
+                    // fails destination tests
+                    .put("HKD", 10.1)
+                    .put("NZD", 700.1)
                     .build()))),
         new AirbyteMessage()
             .withType(Type.STATE)
@@ -470,8 +474,10 @@ public void testIncrementalSync() throws Exception {
                     .put("id", 1)
                     .put("currency", "USD")
                     .put("date", "2020-03-31T00:00:00Z")
-                    .put("HKD", 10.0)
-                    .put("NZD", 700.0)
+                    // TODO(sherifnada) hack: write decimals with sigfigs because Snowflake stores 10.1 as "10" which
+                    // fails destination tests
+                    .put("HKD", 10.1)
+                    .put("NZD", 700.1)
                     .build()))),
         new AirbyteMessage()
             .withType(Type.STATE)

diff --git a/airbyte-integrations/connectors/destination-snowflake/Dockerfile b/airbyte-integrations/connectors/destination-snowflake/Dockerfile
@@ -8,5 +8,5 @@ COPY build/distributions/${APPLICATION}*.tar ${APPLICATION}.tar
 
 RUN tar xf ${APPLICATION}.tar --strip-components=1
 
-LABEL io.airbyte.version=0.3.9
+LABEL io.airbyte.version=0.3.10
 LABEL io.airbyte.name=airbyte/destination-snowflake
diff --git a/...wflake/src/main/java/io/airbyte/integrations/destination/snowflake/SnowflakeDatabase.java b/...wflake/src/main/java/io/airbyte/integrations/destination/snowflake/SnowflakeDatabase.java
@@ -59,6 +59,10 @@ public static Connection getConnection(JsonNode config) throws SQLException {
     // allows queries to contain any number of statements.
     properties.put("MULTI_STATEMENT_COUNT", 0);
 
+    // https://docs.snowflake.com/en/user-guide/jdbc-parameters.html#application
+    // identify airbyte traffic to snowflake to enable partnership & optimization opportunities
+    properties.put("application", "airbyte");
+
     return DriverManager.getConnection(connectUrl, properties);
   }
 

diff --git a/docs/integrations/destinations/snowflake.md b/docs/integrations/destinations/snowflake.md
@@ -186,3 +186,8 @@ The final query should show a `STORAGE_GCP_SERVICE_ACCOUNT` property with an ema
 
 Finally, you need to add read/write permissions to your bucket with that email.
 
+
+| Version | Date       | Pull Request | Subject |
+| :------ | :--------  | :-----       | :------ |
+| 0.3.10 | July 12, 2021| [4713](https://github.com/airbytehq/airbyte/pull/4713)| Tag traffic with `airbyte` label to enable optimization opportunities from Snowflake |
+
diff --git a/docs/integrations/sources/stripe.md b/docs/integrations/sources/stripe.md
@@ -69,4 +69,4 @@ If you would like to test Airbyte using test data on Stripe, `sk_test_` and `rk_
 | 0.1.11   | 2021-05-30 | [3744](https://github.com/airbytehq/airbyte/pull/3744) | Fix types in schema |
 | 0.1.10   | 2021-05-28 | [3728](https://github.com/airbytehq/airbyte/pull/3728) | Update data types to be number instead of int |
 | 0.1.9   | 2021-05-13 | [3367](https://github.com/airbytehq/airbyte/pull/3367) | Add acceptance tests for connected accounts |
-| 0.1.8   | 2021-05-11 | [3566](https://github.com/airbytehq/airbyte/pull/3368) | Bump CDK connectors |
+| 0.1.8   | 2021-05-11 | [3566](https://github.com/airbytehq/airbyte/pull/3368) | Bump CDK connectors |