[GitHub] [arrow] jorisvandenbossche commented on a change in pull request #8661: ARROW-10581: [website] IPC dictionary reference to relevant section

[GitBox <gi...@apache.org>]

jorisvandenbossche commented on a change in pull request #8661:
URL: https://github.com/apache/arrow/pull/8661#discussion_r526136508



##########
File path: docs/source/format/Columnar.rst
##########
@@ -1053,7 +1053,7 @@ the dictionaries can be properly interpreted: ::
 
 The dictionary ``id`` in the message metadata can be referenced one or more times
 in the schema, so that dictionaries can even be used for multiple fields. See
-the :doc:`Layout` document for more about the semantics of
+the `Dictionary-encoded Layout`_ section for more about the semantics of

Review comment:
       Could you add a label to that section and use `` :ref`..` `` syntax instead? That's more robust in case the title is changed (see https://www.sphinx-doc.org/en/master/usage/restructuredtext/roles.html#cross-referencing-arbitrary-locations, and its recommendation to use that instead of standard links)




----------------------------------------------------------------
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:
users@infra.apache.org