Merge pull request #5824 from jakirkham/mark_host_frames_readonly

Mark host frames as not needing to be writeable
rapidsai · Aug 4, 2020 · a4802bc · a4802bc
2 parents 36bb50a + 03000e9
commit a4802bc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -89,6 +89,7 @@
 - PR #5373 Remove legacy nvstrings/nvcategory/nvtext
 - PR #5362 Remove dependency on `rmm._DevicePointer`
 - PR #5302 Add missing comparison operators to `fixed_point` type
+- PR #5824 Mark host frames as not needing to be writeable
 - PR #5354 Split Dask deserialization methods by dask/cuda
 - PR #5363 Handle `0-dim` inputs while broadcasting to a column
 - PR #5396 Remove legacy tests env variable from build.sh

diff --git a/python/cudf/cudf/core/abc.py b/python/cudf/cudf/core/abc.py
@@ -95,6 +95,7 @@ def host_serialize(self):
         :meta private:
         """
         header, frames = self.device_serialize()
+        header["writeable"] = len(frames) * (None,)
         frames = [
             f.to_host_array().data if c else memoryview(f)
             for c, f in zip(header["is-cuda"], frames)