You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Krisztian Szucs (Jira)" <ji...@apache.org> on 2022/04/21 18:20:00 UTC
[jira] [Assigned] (ARROW-16158) [C++] rename ARROW_ENGINE to ARROW_SUBSTRAIT
[ https://issues.apache.org/jira/browse/ARROW-16158?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Krisztian Szucs reassigned ARROW-16158:
---------------------------------------
Assignee: Krisztian Szucs
> [C++] rename ARROW_ENGINE to ARROW_SUBSTRAIT
> --------------------------------------------
>
> Key: ARROW-16158
> URL: https://issues.apache.org/jira/browse/ARROW-16158
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Jonathan Keane
> Assignee: Krisztian Szucs
> Priority: Blocker
> Labels: pull-request-available
> Fix For: 8.0.0
>
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> When we introduced substrait we reused the cmake + feature {{ARROW_ENGINE}} to mean compute+a few other things as well as the substrait consumer functionality. In general, right now, we don't yet need (or want) to build substrait in our packages (e.g. the R package) since many places don't yet take advantage of it. But the naming of the cmake or feature is now confusing: it effectively is only substrait if you separately enable compute, etc. but it makes it sound like the query engine we have been building since 6.0.0 is disabled.
> We should rename {{ARROW_ENGINE}} to {{ARROW_SUBSTRAIT}} now and then we can add an {{ARROW_ENGINE}} later if we need to encompass a larger set of engine functionality (e.g. compute+spillover+scheduler+memory limits) if that's needed.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)