make mapping validation too laxist rather than too strict

alkemics · Jul 5, 2021 · 14a66f6 · 14a66f6
1 parent 159f204
commit 14a66f6
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pandagg/node/aggs/abstract.py b/pandagg/node/aggs/abstract.py
@@ -98,7 +98,9 @@ def valid_on_field_type(cls, field_type):
             return field_type in cls.WHITELISTED_MAPPING_TYPES
         if cls.BLACKLISTED_MAPPING_TYPES is not None:
             return field_type not in cls.BLACKLISTED_MAPPING_TYPES
-        return False
+        # by default laxist
+        # TODO - constraint to only allowed types
+        return True
 
     def to_dict(self):
         """