VELOX_MONO is commonly required for static build from source, please don't remove it.

[MBkkt]

VELOX_MONO is commonly required for static build from source, please don't remove it.

Because it allows to build and link faster when velox is dependency and also allows to use less disk space for build targets

[assignUser]

Why do you think we would remove it? We just made it the default: #14663

[MBkkt]

Reducing/combining those targets is a longterm goal but wasn't feasible in the short-term, hence the workaround with the mono library.

workaround

[assignUser]

Right, we would only remove it, if we had achieved the goal of reducing the number of targets to a single one. That is probably not something that will happen and so this workaround, like so many, will stay permanently ;)

[MBkkt]

Okay, sounds good to me :)