niyue commented on PR #38763:
URL: https://github.com/apache/arrow/pull/38763#issuecomment-1831848303

   @AlenkaF 
   
   > on the index page list all the subpages with toctree directive (.. 
toctree::)
   > It would be good to maybe move more content from 
[docs/source/cpp/gandiva.rst](https://arrow.apache.org/docs/dev/cpp/gandiva.html)
 to subpages, save the subpages in the gandiva folder and add them to the index 
page toctree
   * I move the three sections in the index page to a new sub page
   * I added the toctree to the index page, but currently there is not too much 
content for gandiva so I still keep the brief summary sections above, and added 
the toctree below. Let me know if this is not desirable.
   
   > The overall flow chart from the PR
   
   I updated the overall flow chart, and remove some internal classes from the 
chart and add external C functions to the chart


-- 
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]

Reply via email to