Fix fixed length binary values

Author: kesmit13

accel.c

@@ -436,6 +436,19 @@ int ucs4_to_utf8(const uint32_t *ucs4_str, size_t ucs4_len, char **utf8_str) {
     return -1;
 }
 
+size_t length_without_trailing_nulls(const char *str, size_t len) {
+    if (!str || len == 0) {
+        return 0; // Handle null or empty input
+    }
+
+    // Start from the end of the string and move backward
+    while (len > 0 && str[len - 1] == '\0') {
+        len--;
+    }
+
+    return len;
+}
+
 //
 // Cached int values for date/time components
 //
@@ -4022,6 +4035,9 @@ static PyObject *dump_rowdat_1_numpy(PyObject *self, PyObject *args, PyObject *k
                     } else {
                         Py_ssize_t str_l = col_types[i].length;
                         CHECKMEM(8+str_l);
+
+                        str_l = length_without_trailing_nulls(bytes, str_l);
+
                         i64 = str_l;
                         memcpy(out+out_idx, &i64, 8);
                         out_idx += 8;

singlestoredb/functions/ext/asgi.py

@@ -723,7 +723,6 @@ async def __call__(
                         func_info['colspec'], b''.join(data),
                     ),
                 )
-                print(func_info['returns'], out)
                 body = output_handler['dump'](
                     [x[1] for x in func_info['returns']], *out,  # type: ignore
                 )

singlestoredb/tests/test_ext_func.py

@@ -1213,9 +1213,9 @@ def test_numpy_fixed_binary(self):
         self.cur.execute('select * from numpy_fixed_binary()')
 
         assert [tuple(x) for x in self.cur] == [
-            ('hello'.encode('utf8') + b'\x00' * 8,),
+            ('hello'.encode('utf8'),),
             ('hi there 😜'.encode('utf8'),),
-            ('😜 bye'.encode('utf8') + b'\x00' * 5,),
+            ('😜 bye'.encode('utf8'),),
         ]
 
         desc = self.cur.description