[jira] [Commented] (ARROW-4261) [C++] CMake paths for IPC, Flight, Thrift, and Plasma don't support using Arrow as a subproject

2019-01-17 Thread Uwe L. Korn (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16745169#comment-16745169 ] Uwe L. Korn commented on ARROW-4261: I agree with [~pitrou] here. We will accept small fixes for this

[jira] [Commented] (ARROW-4261) [C++] CMake paths for IPC, Flight, Thrift, and Plasma don't support using Arrow as a subproject

2019-01-14 Thread Antoine Pitrou (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742509#comment-16742509 ] Antoine Pitrou commented on ARROW-4261: --- I don't think we should promise anything here. However the

[jira] [Commented] (ARROW-4261) [C++] CMake paths for IPC, Flight, Thrift, and Plasma don't support using Arrow as a subproject

2019-01-14 Thread Wes McKinney (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742495#comment-16742495 ] Wes McKinney commented on ARROW-4261: - Interested in opinions of [~xhochy] and [~pitrou]. I can see

[jira] [Commented] (ARROW-4261) [C++] CMake paths for IPC, Flight, Thrift, and Plasma don't support using Arrow as a subproject

2019-01-14 Thread Michael Vilim (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742478#comment-16742478 ] Michael Vilim commented on ARROW-4261: -- Using ExternalProject_Add instead of add_subdirectory makes

[jira] [Commented] (ARROW-4261) [C++] CMake paths for IPC, Flight, Thrift, and Plasma don't support using Arrow as a subproject

2019-01-14 Thread Wes McKinney (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742470#comment-16742470 ] Wes McKinney commented on ARROW-4261: - Hm. I'm not sure that this is a use case that we should invest