You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ko...@apache.org on 2022/12/25 20:37:04 UTC

[arrow] branch master updated: GH-14990: [C++][Skyhook] Follow FileFormat API change (#15086)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 8e9165e83e GH-14990: [C++][Skyhook] Follow FileFormat API change (#15086)
8e9165e83e is described below

commit 8e9165e83eac7acd0137ef40f397dc05cfacd872
Author: Sutou Kouhei <ko...@clear-code.com>
AuthorDate: Mon Dec 26 05:36:53 2022 +0900

    GH-14990: [C++][Skyhook] Follow FileFormat API change (#15086)
    
    `arrow::dataset::FileFormat`'s constructor API was changed in ARROW-17288/GH-14663.
    * Closes: #14990
    
    Authored-by: Sutou Kouhei <ko...@clear-code.com>
    Signed-off-by: Sutou Kouhei <ko...@clear-code.com>
---
 cpp/src/skyhook/client/file_skyhook.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/cpp/src/skyhook/client/file_skyhook.cc b/cpp/src/skyhook/client/file_skyhook.cc
index cc26280353..725b04a345 100644
--- a/cpp/src/skyhook/client/file_skyhook.cc
+++ b/cpp/src/skyhook/client/file_skyhook.cc
@@ -14,6 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
+
 #include "skyhook/client/file_skyhook.h"
 #include "skyhook/protocol/rados_protocol.h"
 #include "skyhook/protocol/skyhook_protocol.h"
@@ -132,7 +133,7 @@ arrow::Result<std::shared_ptr<SkyhookFileFormat>> SkyhookFileFormat::Make(
 
 SkyhookFileFormat::SkyhookFileFormat(std::shared_ptr<RadosConnCtx> ctx,
                                      std::string file_format)
-    : impl_(new Impl(std::move(ctx), std::move(file_format))) {}
+    : FileFormat(nullptr), impl_(new Impl(std::move(ctx), std::move(file_format))) {}
 
 SkyhookFileFormat::~SkyhookFileFormat() = default;