zanmato1984 commented on code in PR #46476: URL: https://github.com/apache/arrow/pull/46476#discussion_r2093538743
########## docs/source/developers/cpp/acero.rst: ########## Review Comment: This file is moved from "docs/source/cpp/acero/developer_guide.rst" with two small tweaks: 1. Changing the title from "Developer’s Guide" to "Developing Acero" (which is more aligned with the naming of other docs at the same level). 2. Adding the "Others" section at the bottom as a TOC to enclose sub-docs which are too elaborate to be inlined (currently only swiss_table.rst). Please note the original "acero.rst" was moved to "acero/swiss_table.rst" (I was thinking about inlining it however found it too long), but git doesn't recognize this pattern and shows many diffs. The actual changes are just the two places mentioned above. ########## docs/source/developers/cpp/acero/swiss_table.rst: ########## Review Comment: As mentioned in other comment, this is renamed from "developers/cpp/acero.rst". ########## docs/source/cpp/acero.rst: ########## Review Comment: Note this is renamed from "streaming_execution.rst" to "acero.rst" which is more explicit (and shorter). Because "Acero" has become a much more well-known name I assume. -- 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]
