Skip to content

Commit

Permalink
fixed type mapping
Browse files Browse the repository at this point in the history
  • Loading branch information
koszti committed Sep 8, 2019
1 parent 2d34b29 commit bfa13e9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion target_redshift/db_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def column_type(schema_property, with_length=True):
column_type = 'character varying'
varchar_length = SHORT_VARCHAR_LENGTH
elif 'number' in property_type:
column_type = 'float'
column_type = 'double precision'
elif 'integer' in property_type and 'string' in property_type:
column_type = 'character varying'
varchar_length = LONG_VARCHAR_LENGTH
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/test_unit.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def test_column_type_mapping(self):
assert mapper(json_dt_or_null) == 'timestamp without time zone'
assert mapper(json_t) == 'character varying(256)'
assert mapper(json_t_or_null) == 'character varying(256)'
assert mapper(json_num) == 'float'
assert mapper(json_num) == 'double precision'
assert mapper(json_int) == 'numeric'
assert mapper(json_int_or_str) == 'character varying(65535)'
assert mapper(json_bool) == 'boolean'
Expand Down

0 comments on commit bfa13e9

Please sign in to comment.