Speedups result (#253)

* One more sep * cdef of "result" Previously, was defining "out" but using "result"
dask · Dec 4, 2017 · 840acec · 840acec
1 parent 2102d0d
commit 840acec
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/fastparquet/speedups.pyx b/fastparquet/speedups.pyx
@@ -48,7 +48,7 @@ def array_encode_utf8(inp):
     cdef:
         Py_ssize_t i, n
         np.ndarray[object] arr
-        np.ndarray[object] out
+        np.ndarray[object] result
 
     arr = _to_array(inp)
     _check_1d_object_array(arr)
@@ -70,7 +70,7 @@ def array_decode_utf8(inp):
     cdef:
         Py_ssize_t i, n
         np.ndarray[object] arr
-        np.ndarray[object] out
+        np.ndarray[object] result
         object val
 
     arr = _to_array(inp)

diff --git a/fastparquet/writer.py b/fastparquet/writer.py
@@ -866,6 +866,7 @@ def partition_on_columns(data, columns, root_path, partname, fmd, sep,
     be written in structured directories.
     """
     gb = data.groupby(columns)
+    sep = '/'  # internal paths
     remaining = list(data)
     for column in columns:
         remaining.remove(column)