[C++] Thrift compiler reruns in arrow/dbi/hiveserver2/thrift when using Ninja build

[asfimport]

I've spent some time trying to figure out why this is, playing with different dependency orderings and file properties.

I don't see why this command is fundamentally different from our Flatbuffers-generated files which are basically doing the same thing

https://github.com/apache/arrow/blob/master/cpp/src/arrow/dbi/hiveserver2/thrift/CMakeLists.txt#L65

The code to generate the Thrift bindings originally came from Impala so there might be some small bug. We could take a look if the bug is present in Impala (with Ninja) and see if not, see how what they have is different and if that will fix it

Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney / @wesm

_{Note: This issue was originally created as ARROW-3145. Please see the migration documentation for further details.}

[asfimport]

Wes McKinney / @wesm:
Seems like some kind of "touch" workaround could be employed as a last resort similar to https://stackoverflow.com/questions/41597612/add-custom-command-does-not-re-run-on-failure (not portable to Windows though)

[asfimport]

Wes McKinney / @wesm:
I fixed this in my flight-cpp-prototype branch, will resolve this once that is merged

[asfimport]

Wes McKinney / @wesm:
I fixed this in db0ef22#diff-caf39e9e4747cfd6c97822e76dbac4e5