Merge pull request #118 from dbt-msft/fix_bug_snapshots

Fix for bugs in snapshots

Author: mikaelene

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+### v0.19.0.2
+
+#### fixes
+- solved a bug in snapshots introduced in v0.19.0. Fixes: [#108](https://github.com/dbt-msft/dbt-sqlserver/issues/108), [#117](https://github.com/dbt-msft/dbt-sqlserver/issues/117). 
+
+
 ### v0.19.0.1
 
 #### fixes

dbt/include/sqlserver/macros/adapters.sql

@@ -173,7 +173,7 @@
 {% endmacro %}
 
 {% macro sqlserver__current_timestamp() -%}
-  getdate()
+  SYSDATETIME()
 {%- endmacro %}
 
 {% macro sqlserver__get_columns_in_relation(relation) -%}

dbt/include/sqlserver/macros/materializations/snapshot/strategies.sql

@@ -1,6 +1,6 @@
 {% macro sqlserver__snapshot_hash_arguments(args) %}
     CONVERT(VARCHAR(32), HashBytes('MD5', {% for arg in args %}
-        coalesce(CONVERT(varchar, {{ arg }}, 121), '') {% if not loop.last %} + '|' + {% endif %}
+        coalesce(cast({{ arg }} as varchar(max)), '') {% if not loop.last %} + '|' + {% endif %}
     {% endfor %}), 2)
 {% endmacro %}