westonpace commented on a change in pull request #10008: URL: https://github.com/apache/arrow/pull/10008#discussion_r615108169
########## File path: cpp/src/arrow/dataset/scanner.h ########## @@ -166,13 +169,29 @@ class ARROW_DS_EXPORT ScanTask { std::shared_ptr<Fragment> fragment_; }; -template <typename T> -struct Enumerated { - T value; - int index; - bool last; +/// \brief A trivial ScanTask that yields the RecordBatch of an array. Review comment: Well, for clarity's sake. I just moved them back to where you had them. I"ll leave internal headers for a future PR to keep this one at a reasonable size. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org