Merge pull request wintersrd#65 from s7clarke10/feature/fix_singer_decimal

Supporting different SQL Server info schemas - Singer Decimal

Author: s7clarke10

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 2.2.2
+current_version = 2.2.3
 parse = (?P<major>\d+)
 	\.(?P<minor>\d+)
 	\.(?P<patch>\d+)

CHANGELOG.md

@@ -1,3 +1,7 @@
+# tap-mssql 2.2.3 2024-02-26
+
+* Bug Fix. Enhancing singer-decimal output for numeric data to correctly output the correct datatype - string and precision.
+
 # tap-mssql 2.2.1 2023-10-30
 
 * Bug Fix. Removing test code which slipped into the release, and adjusting offending code with correct dynamic column name.

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "tap-mssql"
-version = "2.2.2"
+version = "2.2.3"
 description = "A pipelinewise compatible tap for connecting Microsoft SQL Server"
 authors = ["Rob Winters <[email protected]>"]
 license = "GNU Affero"

tap_mssql/__init__.py

@@ -122,9 +122,9 @@ def schema_for_column(c, config):
 
     elif data_type in DECIMAL_TYPES:
         if use_singer_decimal:
-            result.type = ["null","number"]
+            result.type = ["null","number","string"]
             result.format = "singer.decimal"
-            result.additionalProperties = {"scale_precision": f"({c.character_maximum_length},{c.numeric_scale})"}
+            result.additionalProperties = {"scale_precision": f"({c.character_maximum_length or c.numeric_precision},{c.numeric_scale})"}
         else:
             result.type = ["null", "number"]
             result.multipleOf = 10 ** (0 - c.numeric_scale)