bkietz commented on a change in pull request #8894:
URL: https://github.com/apache/arrow/pull/8894#discussion_r551578256
##########
File path: cpp/src/arrow/compute/cast.cc
##########
@@ -118,8 +118,86 @@ class CastMetaFunction : public MetaFunction {
} // namespace
+const FunctionDoc struct_doc{"Wrap Arrays into a StructArray",
+ ("Names of the StructArray's fields are\n"
+ "specified through StructOptions."),
+ {},
+ "StructOptions"};
+
+Result<ValueDescr> StructResolve(KernelContext* ctx,
+ const std::vector<ValueDescr>& descrs) {
+ const auto& names = OptionsWrapper<StructOptions>::Get(ctx).field_names;
+ if (names.size() != descrs.size()) {
+ return Status::Invalid("Struct() was passed ", names.size(), " field ",
"names but ",
+ descrs.size(), " arguments");
+ }
+
+ size_t i = 0;
+ FieldVector fields(descrs.size());
+
+ ValueDescr::Shape shape = ValueDescr::SCALAR;
+ for (const ValueDescr& descr : descrs) {
+ if (descr.shape != ValueDescr::SCALAR) {
+ shape = ValueDescr::ARRAY;
+ } else {
+ switch (descr.type->id()) {
+ case Type::EXTENSION:
Review comment:
ExtensionScalar hasn't been implemented yet
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
