division by zero error

Author: joocer

opteryx/__version__.py

@@ -1,9 +1,9 @@
 # THIS FILE IS AUTOMATICALLY UPDATED DURING THE BUILD PROCESS
 # DO NOT EDIT THIS FILE DIRECTLY
 
-__build__ = 1674
+__build__ = 1675
 __author__ = "@joocer"
-__version__ = "0.26.0-beta.1674"
+__version__ = "0.26.0-beta.1675"
 
 # Store the version here so:
 # 1) we don't load dependencies by storing it in __init__.py

opteryx/compiled/structures/memory_pool.pyx

@@ -110,10 +110,10 @@ cdef class MemoryPool:
             self._print_stats()
 
     cdef void _print_stats(self):
-        cdef int64_t total_free = 0
-        cdef int64_t total_used = 0
-        cdef int64_t fragmentation = 0
-        cdef int64_t free_blocks = 0
+    cdef int64_t total_free = 0
+    cdef int64_t total_used = 0
+    cdef double fragmentation = 0.0
+    cdef int64_t free_blocks = 0
 
         for i in range(self.segments.size()):
             if self.segments[i].is_free:
@@ -123,7 +123,12 @@ cdef class MemoryPool:
                 total_used += self.segments[i].length
 
         if total_free > 0 and free_blocks > 1:
-            fragmentation = (free_blocks - 1) * 100 / (total_free / 1024)  # Simplified fragmentation metric
+            # Avoid integer division by zero when total_free < 1024.
+            cdef double denom = total_free / 1024.0
+            if denom > 0.0:
+                fragmentation = (free_blocks - 1) * 100.0 / denom  # Simplified fragmentation metric
+            else:
+                fragmentation = 0.0
 
         print(f"Memory Pool ({self.name}) <"
               f"size={self.size}, used={total_used}, free={total_free}, "

opteryx/connectors/disk_connector.py

@@ -157,16 +157,14 @@ def read_blob(
 
             # On Linux advise the kernel that access will be sequential to improve readahead
             if IS_LINUX:
-                try:
+                # if anything goes wrong, ignore
+                with contextlib.suppress(Exception):
                     libc = ctypes.CDLL("libc.so.6")
                     # MADV_SEQUENTIAL is 2 on Linux, but don't hardcode if available
                     MADV_SEQUENTIAL = 2
                     addr = ctypes.c_void_p(ctypes.addressof(ctypes.c_char.from_buffer(_map)))
                     length = ctypes.c_size_t(size)
                     libc.madvise(addr, length, MADV_SEQUENTIAL)
-                except Exception:
-                    # best-effort: if anything goes wrong, ignore
-                    pass
 
             # pass a memoryview of the mmap to decoders - this makes intent explicit
             # and lets decoders that can accept memoryviews avoid extra copies

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "opteryx"
-version = "0.26.0-beta.1674"
+version = "0.26.0-beta.1675"
 description = "Query your data, where it lives"
 requires-python = '>=3.11'
 readme = {file = "README.md", content-type = "text/markdown"}