Try caching methods that check property types

Author: ReubenFrankel

singer_sdk/helpers/_typing.py

@@ -79,6 +79,7 @@ def append_type(type_dict: dict, new_type: str) -> dict:
     return result
 
 
+@lru_cache
 def is_secret_type(type_dict: dict) -> bool:
     """Return True if JSON Schema type definition appears to be a secret.
 
@@ -109,6 +110,7 @@ def is_secret_type(type_dict: dict) -> bool:
     return False
 
 
+@lru_cache
 def is_object_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is an object or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema:
@@ -122,6 +124,7 @@ def is_object_type(property_schema: dict) -> bool | None:
     )
 
 
+@lru_cache
 def is_uniform_list(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is an array with a single schema.
 
@@ -136,6 +139,7 @@ def is_uniform_list(property_schema: dict) -> bool | None:
     )
 
 
+@lru_cache
 def is_datetime_type(type_dict: dict) -> bool:
     """Return True if JSON Schema type definition is a 'date-time' type.
 
@@ -155,6 +159,7 @@ def is_datetime_type(type_dict: dict) -> bool:
     raise ValueError(msg)
 
 
+@lru_cache
 def is_date_or_datetime_type(type_dict: dict) -> bool:
     """Return True if JSON Schema type definition is a 'date'/'date-time' type.
 
@@ -186,6 +191,7 @@ def is_date_or_datetime_type(type_dict: dict) -> bool:
     raise ValueError(msg)
 
 
+@lru_cache
 def get_datelike_property_type(property_schema: dict) -> str | None:
     """Return one of 'date-time', 'time', or 'date' if property is date-like.
 
@@ -200,6 +206,7 @@ def get_datelike_property_type(property_schema: dict) -> str | None:
     return None
 
 
+@lru_cache
 def _is_string_with_format(type_dict: dict[str, t.Any]) -> bool | None:
     if "string" in type_dict.get("type", []) and type_dict.get("format") in {
         "date-time",
@@ -236,6 +243,7 @@ def handle_invalid_timestamp_in_record(
     raise ValueError(msg)
 
 
+@lru_cache
 def is_string_array_type(type_dict: dict) -> bool:
     """Return True if JSON Schema type definition is a string array."""
     if not type_dict:
@@ -254,6 +262,7 @@ def is_string_array_type(type_dict: dict) -> bool:
     return "array" in type_dict["type"] and bool(is_string_type(type_dict["items"]))
 
 
+@lru_cache
 def is_array_type(type_dict: dict) -> bool:
     """Return True if JSON Schema type is an array."""
     if not type_dict:
@@ -272,6 +281,7 @@ def is_array_type(type_dict: dict) -> bool:
     return "array" in type_dict["type"]
 
 
+@lru_cache
 def is_boolean_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is a boolean or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema:
@@ -287,6 +297,7 @@ def is_boolean_type(property_schema: dict) -> bool | None:
     return False
 
 
+@lru_cache
 def _is_exclusive_boolean_type(property_schema: dict) -> bool:
     if "type" not in property_schema:
         return False
@@ -298,6 +309,7 @@ def _is_exclusive_boolean_type(property_schema: dict) -> bool:
     )
 
 
+@lru_cache
 def is_integer_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is an integer or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema:
@@ -313,6 +325,7 @@ def is_integer_type(property_schema: dict) -> bool | None:
     return False
 
 
+@lru_cache
 def is_string_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is a string or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema:
@@ -328,6 +341,7 @@ def is_string_type(property_schema: dict) -> bool | None:
     return False
 
 
+@lru_cache
 def is_null_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is a null or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema:
@@ -343,6 +357,7 @@ def is_null_type(property_schema: dict) -> bool | None:
     return False
 
 
+@lru_cache
 def is_number_type(property_schema: dict) -> bool | None:
     """Return true if the JSON Schema type is a number or None if detection fails."""
     if "anyOf" not in property_schema and "type" not in property_schema: