Commits
Wes McKinney authored ee928d2233d
ARROW-1211: [C++] Enable builder classes to automatically use the default memory pool I added the `ARROW_NO_DEFAULT_MEMORY_POOL` define option to disable this in third party use. I also flipped the order of arguments to the builder constructors to be a bit more natural. I don't feel strongly about this, but it does make the code a bit nicer: ```c++ FixedSizeBinaryBuilder builder(type); FixedSizeBinaryBuilder builder(type, pool); ``` versus ```c++ FixedSizeBinaryBuilder builder(type); FixedSizeBinaryBuilder builder(pool, type); ``` Author: Wes McKinney <wes.mckinney@twosigma.com> Closes #923 from wesm/ARROW-1211 and squashes the following commits: e8a129d [Wes McKinney] Use CONFIGURE_OPTIONS as advised 7835e67 [Wes McKinney] Build c_glib in Travis CI without deprecated Arrow APIs d86a6a3 [Wes McKinney] Remove copy-paste error, unneeded using statement a712445 [Wes McKinney] Benchmark fixes 1488bb4 [Wes McKinney] Some MSVC fixes, cannot get DictionaryBuilder default ctor working there 33cd7d5 [Wes McKinney] Use macro for memory pool argument to avoid code duplication. Add macro expansion to Doxyfile 085ca86 [Wes McKinney] Add option builder constructors to use default memory pool without passing explicitly. Add ARROW_NO_DEFAULT_MEMORY_POOL define. Flip builder constructor order, deprecate old constructors. README documentation