pitrou commented on code in PR #14495:
URL: https://github.com/apache/arrow/pull/14495#discussion_r1015560078
##########
cpp/src/arrow/compute/api_scalar.h:
##########
@@ -277,12 +277,16 @@ class ARROW_EXPORT SetLookupOptions : public
FunctionOptions {
class ARROW_EXPORT StructFieldOptions : public FunctionOptions {
public:
explicit StructFieldOptions(std::vector<int> indices);
+ explicit StructFieldOptions(std::initializer_list<int>);
+ explicit StructFieldOptions(FieldRef field_ref,
+ std::vector<int> indices = std::vector<int>());
StructFieldOptions();
static constexpr char const kTypeName[] = "StructFieldOptions";
/// The child indices to extract. For instance, to get the 2nd child
/// of the 1st child of a struct or union, this would be {0, 1}.
std::vector<int> indices;
+ FieldRef field_ref;
Review Comment:
Sorry @milesgranger . I think we should simply remove this member. But we
should keep the corresponding constructor.
--
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.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
