westonpace commented on a change in pull request #11556:
URL: https://github.com/apache/arrow/pull/11556#discussion_r741556380
##########
File path: cpp/src/arrow/table.h
##########
@@ -208,6 +208,15 @@ class ARROW_EXPORT Table {
Result<std::shared_ptr<Table>> CombineChunks(
MemoryPool* pool = default_memory_pool()) const;
+ /// \brief Make a new record batch by combining the chunks this table has.
+ ///
+ /// All the underlying chunks in the ChunkedArray of each column are
+ /// concatenated into a single chunk.
+ ///
+ /// \param[in] pool The pool for buffer allocations
+ Result<std::shared_ptr<RecordBatch>> CombineChunksToBatch(
Review comment:
There is already a ConcatenateTables which combines multiple tables into
a single tables. Also, there is already a CombineChunks which concatenates the
underlying arrays but leaves it as a table. Since the behavior is more similar
to the latter I went with that naming scheme.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
