GH-44563: [C++] FunctionOptions::{Serialize,Deserialize}() return an error without ARROW_IPC

[kou]

Rationale for this change

They use IPC format internally. So we can't do them without ARROW_IPC.

What changes are included in this PR?

Returns arrow::Status::NotImplemented without ARROW_IPC.

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

• GitHub Issue: [C++] Static linking with IPC disabled leads to link error in computer/function_internal.cc #44563

[kou]

We don't need to update our tests because ARROW_BUILD_TESTS enables ARROW_IPC implicitly:

arrow/cpp/cmake_modules/DefineOptions.cmake

Lines 219 to 224 in 3752109

	define_option(ARROW_BUILD_TESTS
	"Build the Arrow googletest unit tests"
	OFF
	DEPENDS
	ARROW_IPC
	ARROW_TESTING)

[kou]

I'll merge this tomorrow for 19.0.0 if nobody objects this.

[kou]

Thanks for approval. I'll merge this now.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit ada8750.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 7 possible false positives for unstable benchmarks that are known to sometimes produce them.