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;