Not all headers protected from Windows min/max defines

[abellgithub]

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

Not all headers that use min and max include arrow/util/windows_compatibility.h . This requires a user of arrow to disable these macros on Windows. As an example, line 748 of arrow/type.h uses std::numeric_limits<int32_t>::max(). This can also be fixed by disabling macro expansion (for example, (std::numeric_limits<int32_t>::max)() ) or the Windows macros can be disabled by defining NOMINMAX, which is done in windows_compatibility.h.

Component(s)

C++

[kou]

(std::numeric_limits<int32_t>::max)() is better because it's not depend on the global NOMINMAX definition.
Could you open a pull request?

[abellgithub]

Yes, I'll open a PR.