[jira] [Commented] (ARROW-3974) [C++] Combine field_builders_ and children_ members in array/builder.h

2018-12-11 Thread Wes McKinney (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-3974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16718146#comment-16718146 ] Wes McKinney commented on ARROW-3974: - I'm refactoring builder.h a bit. I'll take car

[jira] [Commented] (ARROW-3974) [C++] Combine field_builders_ and children_ members in array/builder.h

2018-12-11 Thread Wes McKinney (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-3974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16718121#comment-16718121 ] Wes McKinney commented on ARROW-3974: - It's probably easiest to eliminate the {{field

[jira] [Commented] (ARROW-3974) [C++] Combine field_builders_ and children_ members in array/builder.h

2018-12-11 Thread Benjamin Kietzman (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-3974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16717903#comment-16717903 ] Benjamin Kietzman commented on ARROW-3974: -- Which of those is the cruft? > [C++