[ https://issues.apache.org/jira/browse/ARROW-100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15238717#comment-15238717 ]
Micah Kornfield commented on ARROW-100: --------------------------------------- hmm what version, of clang-tidy do you have installed? I thought I fixed this issue (at least for clang-tidy-3.7) when I checked in the initial clang-tidy check. It looks like it is following the path that the assertion should checking for, but it doesn't realize that FatalLogger never returns after destruction. > [C++] Computing RowBatch size > ----------------------------- > > Key: ARROW-100 > URL: https://issues.apache.org/jira/browse/ARROW-100 > Project: Apache Arrow > Issue Type: New Feature > Components: C++ > Reporter: Philipp Moritz > > Hi, > thank you guys for this project, I'm really enjoying what I've seen so far! > There is an unimplemented method for getting the total size of objects: > int64_t GetRowBatchSize(const RowBatch* batch); > Has somebody already started to implement it or thought about how to do it? > It could be done by recursively adding up all the involved buffer sizes, > build the metadata and add its size. Let me know if you want me to create a > draft of the implementation. > -- Philipp. -- This message was sent by Atlassian JIRA (v6.3.4#6332)