ORC-1937: Use the default buildtype in Meson config

[WillAyd]

What changes were proposed in this pull request?

This removes setting the Meson configuration to remove the buildtype of release by default.

Why are the changes needed?

While coming from CMake it may seem like a good idea to build release builds by default, the setting is either ignored or problematic when orc is used as a subproject in other projects, especially on Windows where a main project building a library with debug symbols may fail if a subproject tries to force release symbols.

How was this patch tested?

Compiled locally

Was this patch authored or co-authored using generative AI tooling?

No

[dongjoon-hyun]

Well, I'm a little reluctant to downgrade the default behavior here.

Does Mesos has no corresponding feature like RELWITHDEBINFO of CMake?

[WillAyd]

Yes it does - that would be the debugoptimized build type

https://mesonbuild.com/Builtin-options.html#core-options

[dongjoon-hyun]

Could you provide some reference what is the standard (or recommended way) in Meson? debug or debugoptimized ?

In addition, I'm wondering if Mesos eco-system is enforcing debug for all libraries or not? To me, it's a little weird that release is not enabled by default in source release.

[WillAyd]

By default meson will produce debug libraries. If you want something else, it is encouraged for the user to specify that from the command line when configuring their project, rather than having a (sub-)project enforce a particular build type. There is some upstream discussion about this in mesonbuild/wrapdb#2208

[dongjoon-hyun]

The upstream discussion looks like a new one (5 days ago) instead of the stabilized one. Let's have more time to think about this since we still have enough time for Apache ORC 2.2 release.

[dongjoon-hyun]

+1, LGTM. I slightly revisited the PR title according to the AS-IS status.