tinybird-toolset 2.4.10__tar.gz → 2.4.13__tar.gz

{tinybird_toolset-2.4.10/src/tinybird_toolset.egg-info → tinybird_toolset-2.4.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tinybird-toolset
-Version: 2.4.10
+Version: 2.4.13
 Home-page: https://gitlab.com/tinybird/clickhouse-toolset
 Author: Tinybird.co
 Author-email: support@tinybird.co

{tinybird_toolset-2.4.10 → tinybird_toolset-2.4.13}/conf.py

@@ -32,6 +32,24 @@ MSAN_ENABLED = bool(os.environ.get('MSAN_ENABLED', ''))  # MemorySanitizer
 TSAN_ENABLED = bool(os.environ.get('TSAN_ENABLED', ''))  # ThreadSanitizer
 THINLTO_FLAG = '-flto=thin'
 
+# Taken from https://github.com/cpp-best-practices/cmake_template/blob/a3971f5b45/cmake/CompilerWarnings.cmake
+CLANG_WARNINGS = [
+    '-Wall',
+    '-Wextra',
+    '-Wshadow',
+    '-Wnon-virtual-dtor',
+    '-Wold-style-cast',
+    '-Wcast-align',
+    '-Wunused',
+    '-Woverloaded-virtual',
+    '-Wpedantic',
+    '-Wconversion',
+    '-Wnull-dereference',
+    '-Wdouble-promotion',
+    '-Wformat=2',
+    '-Wimplicit-fallthrough',
+]
+
 
 def apply_patch(patch_path):
     if OMIT_PATCHES:
@@ -276,10 +294,13 @@ class CustomBuildWithFromCH(build_ext):
     def cflags():
         cflags_list = CustomBuildWithFromCH.ninja_extractor_other('FLAGS')
         include_list = list(map(transform_path, CustomBuildWithFromCH.ninja_extractor_other('INCLUDES')))
-        # Reduce the noise coming from python since those flags are added later as -I and not -isystem
-        other_list = ['-Wno-reserved-identifier', '-Wno-zero-as-null-pointer-constant', '-Wno-old-style-cast',
-                      '-Wno-cast-function-type', '-Wno-unused-function', '-Wno-redundant-decls',
-                      '-Wno-missing-include-dirs']
+        # Replace -I with -isystem: these are external headers and we don't want to see warnings from them
+        include_list = [flag.replace('-I', '-isystem', 1) if flag.startswith('-I') else flag for flag in include_list]
+        # Remove -W flags, set our own warnings
+        cflags_list = [flag for flag in cflags_list if not flag.startswith('-W')]
+        other_list = CLANG_WARNINGS.copy()
+        other_list += ['-Werror']  # Treat warnings as errors
+
         if BUILD_FOR_VALGRIND:
             other_list += ['-g', '-gdwarf-4']
         elif DEBUG_SYMBOLS:
@@ -332,6 +353,10 @@ class CustomBuildWithFromCH(build_ext):
 
         # Remove -Wl,--no-undefined since Python symbols will undefined
         link_flags = filter(lambda flag: flag.find('--no-undefined') == -1, link_flags)
+        if sys.platform == 'darwin':
+            # -undefined dynamic_lookup (added in extra_libs) already permits all undefined
+            # symbols, so an explicit -Wl,-U,<symbol> is redundant and makes ld emit a warning.
+            link_flags = filter(lambda flag: not flag.startswith('-Wl,-U,'), link_flags)
         return " ".join(link_flags)
 
     @staticmethod
@@ -384,7 +409,11 @@ class CustomBuildWithFromCH(build_ext):
         linker = getattr(self.compiler, "linker_so_cxx")
         linker = [COMPILER_CC] + split_quoted(self.ldflags()) + linker[1:]
 
-        compiler += ['-isystem', TOOLSET_PATH]
+        compiler += ['-I', TOOLSET_PATH]
+        # Make include_dirs (Python headers; set by setuptools) to -isystem: prevents warnings from Python headers
+        for d in self.include_dirs or []:
+            compiler += ['-isystem', d]
+        self.include_dirs = []
 
         self.compiler.set_executable("compiler_so", compiler)
         self.compiler.set_executable("compiler_so_cxx", compiler)

{tinybird_toolset-2.4.10 → tinybird_toolset-2.4.13}/setup.py

@@ -1,7 +1,7 @@
 from setuptools import setup, Extension
 
 NAME = 'tinybird-toolset'
-VERSION = '2.4.10'
+VERSION = '2.4.13'
 
 try:
     from conf import *

{tinybird_toolset-2.4.10 → tinybird_toolset-2.4.13/src/tinybird_toolset.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tinybird-toolset
-Version: 2.4.10
+Version: 2.4.13
 Home-page: https://gitlab.com/tinybird/clickhouse-toolset
 Author: Tinybird.co
 Author-email: support@tinybird.co

{tinybird_toolset-2.4.10 → tinybird_toolset-2.4.13}/tests/test_convert_to_row_binary.py

@@ -5134,6 +5134,43 @@ TEST_CASES = [
         b'\x00\x00L-100000000000000000000000000000000000000000000000000000000000000000000000000\x00\x01\x1dInvalid decimal string format\x00\x01\x05value\x01\x01',
         0, 1),
 
+    # Booleans imported into Decimal must produce EXACTLY 1.0 and 0.0, without precision loss
+    (ConversionMode.ONLY_LEGACY,
+        [('t', 'Decimal32(4)', '$.t', False), ('f', 'Decimal32(4)', '$.f', False)],
+        '{"t":true,"f":false}',
+        b'\x00\x10\x27\x00\x00\x00\x00\x00\x00\x00',
+        b'',
+        1, 0),
+
+    (ConversionMode.ONLY_LEGACY,
+        [('t', 'Decimal64(8)', '$.t', False), ('f', 'Decimal64(8)', '$.f', False)],
+        '{"t":true,"f":false}',
+        b'\x00\x00\xe1\xf5\x05\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00',
+        b'',
+        1, 0),
+
+    (ConversionMode.ONLY_LEGACY,
+        [('t', 'Decimal128(30)', '$.t', False), ('f', 'Decimal128(30)', '$.f', False)],
+        '{"t":true,"f":false}',
+        b'\x00\x00\x00\x00\x40\xea\xed\x74\x46\xd0\x9c\x2c\x9f\x0c\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00',
+        b'',
+        1, 0),
+
+    (ConversionMode.ONLY_LEGACY,
+        [('t', 'Decimal256(50)', '$.t', False), ('f', 'Decimal256(50)', '$.f', False)],
+        '{"t":true,"f":false}',
+        b'\x00\x00\x00\x00\x00\x00\x00\x64\xb5\xfd\x34\x05\xc4\xd2\x87\x66\x92\xf9\x15\x3b\x6c\x44\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00',
+        b'',
+        1, 0),
+
+    # Non-legacy mode rejects booleans imported into Decimal columns: the row is quarantined.
+    (ConversionMode.ONLY_NONLEGACY,
+        [('value', 'Decimal128(30)', '$.value', False)],
+        '{"value": true}',
+        b'',
+        b"\x00\x00\x04true\x00\x01XInvalid value true for type 'Decimal(38, 30)' in column 'value' with jsonpath '$.value'.\x00\x01\x05value\x01\x01",
+        0, 1),
+
     (ConversionMode.ALWAYS,
         [('value', 'FixedString(6)', '$.value', False)],
         '{}',

{tinybird_toolset-2.4.10 → tinybird_toolset-2.4.13}/tests/test_replace_tables_backward_compat.py

@@ -370,14 +370,16 @@ def _log_cases(cases):
 
 def _results_match(baseline, target, function_name=None):
     # Allow specific functions to transition from blocked to allowed
-    ALLOWED_UNBLOCKS = {"getSetting"}
+    ALLOWED_UNBLOCKS = {"getSetting", "hasAllTokens", "hasAnyTokens", "hasPhrase"}
 
     if baseline.get("ok") != target.get("ok"):
         # Check if this is an allowed unblock (was blocked, now allowed)
         is_unblock = (not baseline.get("ok") and target.get("ok"))
         is_allowed = function_name in ALLOWED_UNBLOCKS
-        is_restriction = "is restricted" in baseline.get("error_message", "")
-        if is_unblock and is_allowed and is_restriction:
+        # Blocked via deny list ("is restricted") or absent from allowlist ("Unknown function ...").
+        baseline_error = baseline.get("error_message", "")
+        is_block = "is restricted" in baseline_error or "Unknown function" in baseline_error
+        if is_unblock and is_allowed and is_block:
             return True  # This is an expected breaking change
         return False
     if baseline.get("ok"):