Rename JSONType → Type for brevity

We know that all these classes are column types, there is no need to specify that.

Also to consistency with other column types.

Author: Sergey Vasilyev

data_diff/hashdiff_tables.py

@@ -7,7 +7,7 @@
 
 from runtype import dataclass
 
-from data_diff.sqeleton.abcs import ColType_UUID, NumericType, PrecisionType, StringType, Boolean, JSONType
+from data_diff.sqeleton.abcs import ColType_UUID, NumericType, PrecisionType, StringType, Boolean, JSON
 
 from .info_tree import InfoTree
 from .utils import safezip, diffs_are_equiv_jsons
@@ -205,7 +205,7 @@ def _bisect_and_diff_segments(
         if max_rows < self.bisection_threshold or max_space_size < self.bisection_factor * 2:
             rows1, rows2 = self._threaded_call("get_values", [table1, table2])
             json_cols = {i: colname for i, colname in enumerate(table1.extra_columns)
-                         if isinstance(table1._schema[colname], JSONType)}
+                         if isinstance(table1._schema[colname], JSON)}
             diff = list(diff_sets(rows1, rows2, json_cols))
 
             info_tree.info.set_diff(diff)

data_diff/sqeleton/abcs/__init__.py

@@ -10,6 +10,6 @@
     PrecisionType,
     StringType,
     Boolean,
-    JSONType,
+    JSON,
 )
 from .compiler import AbstractCompiler, Compilable

data_diff/sqeleton/abcs/database_types.py

@@ -134,19 +134,19 @@ class Text(StringType):
     supported = False
 
 
-class JSONType(ColType):
+class JSON(ColType):
     pass
 
 
-class RedShiftSuper(JSONType):
+class RedShiftSuper(JSON):
     pass
 
 
-class PostgresqlJSON(JSONType):
+class PostgresqlJSON(JSON):
     pass
 
 
-class PostgresqlJSONB(JSONType):
+class PostgresqlJSONB(JSON):
     pass
 
 

data_diff/sqeleton/abcs/mixins.py

@@ -1,5 +1,5 @@
 from abc import ABC, abstractmethod
-from .database_types import TemporalType, FractionalType, ColType_UUID, Boolean, ColType, String_UUID, JSONType
+from .database_types import TemporalType, FractionalType, ColType_UUID, Boolean, ColType, String_UUID, JSON
 from .compiler import Compilable
 
 
@@ -49,7 +49,7 @@ def normalize_uuid(self, value: str, coltype: ColType_UUID) -> str:
             return f"TRIM({value})"
         return self.to_string(value)
 
-    def normalize_json(self, value: str, _coltype: JSONType) -> str:
+    def normalize_json(self, value: str, _coltype: JSON) -> str:
         """Creates an SQL expression, that converts 'value' to its minified json string representation."""
         raise NotImplementedError()
 
@@ -77,7 +77,7 @@ def normalize_value_by_type(self, value: str, coltype: ColType) -> str:
             return self.normalize_uuid(value, coltype)
         elif isinstance(coltype, Boolean):
             return self.normalize_boolean(value, coltype)
-        elif isinstance(coltype, JSONType):
+        elif isinstance(coltype, JSON):
             return self.normalize_json(value, coltype)
         return self.to_string(value)
 

data_diff/sqeleton/databases/base.py

@@ -35,7 +35,7 @@
     DbTime,
     DbPath,
     Boolean,
-    JSONType
+    JSON
 )
 from ..abcs.mixins import Compilable
 from ..abcs.mixins import (
@@ -260,7 +260,7 @@ def parse_type(
         elif issubclass(cls, (Text, Native_UUID)):
             return cls()
 
-        elif issubclass(cls, JSONType):
+        elif issubclass(cls, JSON):
             return cls()
 
         raise TypeError(f"Parsing {type_repr} returned an unknown type '{cls}'.")

data_diff/utils.py

@@ -160,7 +160,7 @@ def diffs_are_equiv_jsons(diff: list, json_cols: dict):
         return False, overriden_diff_cols
     match = True
     for i, (col_a, col_b) in enumerate(safezip(diff[0][1][1:], diff[1][1][1:])):  # index 0 is extra_columns first elem
-        # we only attempt to parse columns of JSONType, but we still need to check if non-json columns don't match
+        # we only attempt to parse columns of JSON type, but we still need to check if non-json columns don't match
         match = col_a == col_b
         if not match and (i in json_cols):
             if _jsons_equiv(col_a, col_b):