You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@arrow.apache.org by "Antoine Pitrou (JIRA)" <ji...@apache.org> on 2018/11/29 15:48:00 UTC

[jira] [Created] (ARROW-3906) [C++] Break builder.cc into multiple compilation units

Antoine Pitrou created ARROW-3906:
-------------------------------------

             Summary: [C++] Break builder.cc into multiple compilation units
                 Key: ARROW-3906
                 URL: https://issues.apache.org/jira/browse/ARROW-3906
             Project: Apache Arrow
          Issue Type: Improvement
          Components: C++
    Affects Versions: 0.11.1
            Reporter: Antoine Pitrou


To improve readability I suggest splitting {{builder.cc}} into independent compilation units. Concrete builder classes are generally independent of each other. The only concern is whether inlining some of the base class implementations is important for performance.

This would also make incremental compilation faster when changing one of the concrete classes.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)