Remove unnecessary stream sync for h_initial_str_offsets

rapidsai · Jan 22, 2025 · 1a34807 · 1a34807
1 parent e838e72
commit 1a34807
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/cpp/src/io/parquet/reader_impl.cpp b/cpp/src/io/parquet/reader_impl.cpp
@@ -415,13 +415,13 @@ void reader::impl::decode_page_data(read_mode mode, size_t skip_rows, size_t num
   // synchronize the streams
   cudf::detail::join_streams(streams, _stream);
 
-  // Sync stream here to use the offsets below
-  auto h_initial_str_offsets = cudf::detail::make_host_vector_sync(initial_str_offsets, _stream);
-
   subpass.pages.device_to_host_async(_stream);
   page_nesting.device_to_host_async(_stream);
   page_nesting_decode.device_to_host_async(_stream);
 
+  // Copy over initial string offsets from device
+  auto h_initial_str_offsets = cudf::detail::make_host_vector_async(initial_str_offsets, _stream);
+
   if (auto const error = error_code.value_sync(_stream); error != 0) {
     CUDF_FAIL("Parquet data decode failed with code(s) " + kernel_error::to_string(error));
   }