This is an automated email from the ASF dual-hosted git repository.

apitrou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new ad26f3ff23 GH-37952: [C++] Make unique->shared explicit to fix build 
failure on at least one compiler (#38136)
ad26f3ff23 is described below

commit ad26f3ff2348292be8d723be64f443553c3703f3
Author: Dewey Dunnington <[email protected]>
AuthorDate: Mon Oct 9 13:31:10 2023 -0300

    GH-37952: [C++] Make unique->shared explicit to fix build failure on at 
least one compiler (#38136)
    
    
    ### Rationale for this change
    
    The 
[test-r-rstudio-r-base-4.1-opensuse153](https://github.com/ursacomputing/crossbow/runs/17162402631)
 nightly has been failing with a compile error for many days.
    
    ### What changes are included in this PR?
    
    Made the unique->shared conversion explicit.
    
    ### Are these changes tested?
    
    Yes (by existing tests)
    
    ### Are there any user-facing changes?
    
    No.
    * Closes: #37952
    
    Authored-by: Dewey Dunnington <[email protected]>
    Signed-off-by: Antoine Pitrou <[email protected]>
---
 cpp/src/arrow/dataset/file_parquet.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/dataset/file_parquet.cc 
b/cpp/src/arrow/dataset/file_parquet.cc
index 3cad1ddd83..9c187f12f6 100644
--- a/cpp/src/arrow/dataset/file_parquet.cc
+++ b/cpp/src/arrow/dataset/file_parquet.cc
@@ -462,7 +462,7 @@ Result<std::shared_ptr<parquet::arrow::FileReader>> 
ParquetFileFormat::GetReader
   std::unique_ptr<parquet::arrow::FileReader> arrow_reader;
   RETURN_NOT_OK(parquet::arrow::FileReader::Make(
       options->pool, std::move(reader), std::move(arrow_properties), 
&arrow_reader));
-  return arrow_reader;
+  return std::move(arrow_reader);
 }
 
 Future<std::shared_ptr<parquet::arrow::FileReader>> 
ParquetFileFormat::GetReaderAsync(

Reply via email to