[C++] Store generated Flight protocol buffer headers in src/generated

apache / arrow

Apache Arrow is the universal columnar format and multi-language toolbox for fast data interchange and in-memory analytics

https://arrow.apache.org/

Apache License 2.0

14.62k stars 3.55k forks source link

[C++] Store generated Flight protocol buffer headers in src/generated #31083

Open asfimport opened 2 years ago

asfimport commented 2 years ago

We strive to store all generated header files (by the Thrift or Flatbuffers compilers, for example) in src/generated. However, it seems that Flight puts its protocol buffers generated files in <CMake build dir>/src/arrow/flight and <CMake build dir>/src/arrow/flight/sql.

It would be nice to eventually harmonize this.

Reporter: Antoine Pitrou / @pitrou

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

asfimport commented 2 years ago

Antoine Pitrou / @pitrou: @lidavidm Feel free to chime in. In any case, it's just a nice-to-have.

asfimport commented 2 years ago

David Li / @lidavidm: Yeah, we should clean things up here.