Skip to content

[C++][FlightSQL] When an error occurs during installation with the '-DARROW_FLIGHT_SQL=ON' option. #38210

@llama90

Description

@llama90

Describe the bug, including details regarding any error messages, version, and platform.

As the title suggests, an error occurs during the install command when including the '-DARROW_FLIGHT_SQL=ON' option after rebasing to the most recent commit.

It seems to be related to the commit(92de9a3). In the code, it's being handled as std::string app_metadata = ""; to set a default value. However, when I run the install, if I do not explicitly set the "" value, the following error occurs.

/Users/lama/workspace/arrow-2/cpp/src/arrow/flight/integration_tests/test_integration.cc:560:80: error: missing field 'app_metadata' initializer [-Werror,-Wmissing-field-initializers]
        {std::to_string(statuses_.size()) + ": " + ticket}, {}, expiration_time});

Adding " " explicitly to app_metadata for all calls to the FlightEndpoint constructor resolves the issue, allowing the command to execute without errors.

The constructor is already written to substitute a default value. Is this approach correct?

Environment

M1(ARM) Macbook

Component(s)

C++

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions