Skip to content

Commit 7922ed0

Browse files
authored
ARROW-16994: [Docs][CI] Clean up docs warnings (apache#13533)
Authored-by: David Li <li.davidm96@gmail.com> Signed-off-by: David Li <li.davidm96@gmail.com>
1 parent 6f95d9d commit 7922ed0

12 files changed

Lines changed: 29 additions & 31 deletions

File tree

.github/workflows/docs_light.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040
name: AMD64 Conda Python 3.9 Sphinx Documentation
4141
runs-on: ubuntu-latest
4242
if: ${{ !contains(github.event.pull_request.title, 'WIP') }}
43-
timeout-minutes: 30
43+
timeout-minutes: 45
4444
env:
4545
PYTHON: "3.9"
4646
steps:

docs/source/cpp/api/compute.rst

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,6 @@ Streaming Execution Operators
7373
Execution Plan Expressions
7474
--------------------------
7575

76-
.. doxygenclass:: arrow::FieldRef
77-
:members:
78-
7976
.. doxygengroup:: expression-core
8077
:content-only:
8178
:members:

docs/source/cpp/api/datatype.rst

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,11 @@ Helpers for looking up fields
106106

107107
.. doxygenclass:: arrow::FieldPath
108108
:members:
109+
:undoc-members:
109110

110111
.. doxygenclass:: arrow::FieldRef
111112
:members:
113+
:undoc-members:
112114

113115

114116
Utilities

docs/source/cpp/api/flight.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,5 +204,5 @@ error codes.
204204
Implementing Custom Transports
205205
==============================
206206

207-
.. doxygenfile:: arrow/flight/transport_impl.h
207+
.. doxygenfile:: arrow/flight/transport.h
208208
:sections: briefdescription detaileddescription

docs/source/cpp/api/support.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ introduced API).
7676
Runtime Configuration
7777
=====================
7878

79-
.. doxygenstruct:: arrow::ArrowGlobalOptions
79+
.. doxygenstruct:: arrow::GlobalOptions
8080
:members:
8181

8282

docs/source/cpp/build_system.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,6 @@ XML. To download, you can use the following batch script:
183183
By default, the timezone database will be detected at ``%USERPROFILE%\Downloads\tzdata``,
184184
but you can set a custom path at runtime in :struct:`arrow::ArrowGlobalOptions`::
185185

186-
arrow::ArrowGlobalOptions options;
187-
options.tz_db_path = "path/to/tzdata";
186+
arrow::GlobalOptions options;
187+
options.timezone_db_path = "path/to/tzdata";
188188
ARROW_RETURN_NOT_OK(arrow::Initialize(options));

docs/source/cpp/examples/row_columnar_conversion.rst

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -33,22 +33,22 @@ instance, and then converts it back to the original array of structs.
3333
Dynamic Schemas
3434
---------------
3535

36-
In many cases, we need to convert to and from row data that does not have a
36+
In many cases, we need to convert to and from row data that does not have a
3737
schema known at compile time. To help implement these conversions, this library
3838
provides several utilities:
3939

40-
* :class:`arrow::RecordBatchBuilder`: creates and manages array builders for
40+
* :class:`arrow::RecordBatchBuilder`: creates and manages array builders for
4141
a full record batch.
4242
* :func:`arrow::VisitTypeInline`: dispatch to functions specialized for the given
4343
array type.
4444
* :ref:`type-traits` (such as ``arrow::enable_if_primitive_ctype``): narrow template
45-
functions to specific Arrow types, useful in conjunction with
45+
functions to specific Arrow types, useful in conjunction with
4646
the :ref:`cpp-visitor-pattern`.
4747
* :class:`arrow::TableBatchReader`: read a table in a batch at a time, with each
4848
batch being a zero-copy slice.
4949

5050
The following example shows how to implement conversion between ``rapidjson::Document``
51-
and Arrow objects. You can read the full code example at
51+
and Arrow objects. You can read the full code example at
5252
https://github.com/apache/arrow/blob/master/cpp/examples/arrow/rapidjson_row_converter.cc
5353

5454
Writing conversions to Arrow
@@ -58,35 +58,35 @@ To convert rows to Arrow record batches, we'll setup Array builders for all the
5858
and then for each field iterate through row values and append to the builders.
5959
We assume that we already know the target schema, which may have
6060
been provided by another system or was inferred in another function. Inferring
61-
the schema *during* conversion is a challenging proposition; many systems will
61+
the schema *during* conversion is a challenging proposition; many systems will
6262
check the first N rows to infer a schema if there is none already available.
6363

6464
At the top level, we define a function ``ConvertToRecordBatch``:
6565

6666
.. literalinclude:: ../../../../cpp/examples/arrow/rapidjson_row_converter.cc
6767
:language: cpp
6868
:start-at: arrow::Result<std::shared_ptr<arrow::RecordBatch>> ConvertToRecordBatch(
69-
:end-at: } // ConvertToRecordBatch
69+
:end-at: } // ConvertToRecordBatch
7070
:linenos:
7171
:lineno-match:
7272

7373
First we use :class:`arrow::RecordBatchBuilder`, which conveniently creates builders
7474
for each field in the schema. Then we iterate over the fields of the schema, get
7575
the builder, and call ``Convert()`` on our ``JsonValueConverter`` (to be discussed
76-
next). At the end, we call ``batch->ValidateFull()``, which checks the integrity
76+
next). At the end, we call ``batch->ValidateFull()``, which checks the integrity
7777
of our arrays to make sure the conversion was performed correctly, which is useful
78-
for debugging new conversion implementations.
78+
for debugging new conversion implementations.
7979

8080
One level down, the ``JsonValueConverter`` is responsible for appending row values
8181
for the provided field to a provided array builder. In order to specialize logic
8282
for each data type, it implements ``Visit`` methods and calls :func:`arrow::VisitTypeInline`.
8383
(See more about type visitors in :ref:`cpp-visitor-pattern`.)
8484

8585
At the end of that class is the private method ``FieldValues()``, which returns
86-
an iterator of the column values for the current field across the rows. In
87-
row-based structures that are flat (such as a vector of values) this may be
86+
an iterator of the column values for the current field across the rows. In
87+
row-based structures that are flat (such as a vector of values) this may be
8888
trivial to implement. But if the schema is nested, as in the case of JSON documents,
89-
a special iterator is needed to navigate the levels of nesting. See the
89+
a special iterator is needed to navigate the levels of nesting. See the
9090
`full example <https://github.com/apache/arrow/blob/master/cpp/examples/arrow/rapidjson_row_converter.cc>`_
9191
for the implementation details of ``DocValuesIterator``.
9292

@@ -100,7 +100,7 @@ for the implementation details of ``DocValuesIterator``.
100100
Writing conversions from Arrow
101101
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
102102

103-
To convert into rows *from* Arrow record batches, we'll process the table in
103+
To convert into rows *from* Arrow record batches, we'll process the table in
104104
smaller batches, visiting each field of the batch and filling the output rows
105105
column-by-column.
106106

@@ -109,7 +109,7 @@ for converting Arrow batches and tables to rows. In many cases, it's more optima
109109
to perform conversions to rows in smaller batches, rather than doing the entire
110110
table at once. So we define one ``ConvertToVector`` method to convert a single
111111
batch, then in the other conversion method we use :class:`arrow::TableBatchReader`
112-
to iterate over slices of a table. This returns Arrow's iterator type
112+
to iterate over slices of a table. This returns Arrow's iterator type
113113
(:class:`arrow::Iterator`) so rows could then be processed either one-at-a-time
114114
or be collected into a container.
115115

docs/source/developers/experimental_repos.rst

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515
.. specific language governing permissions and limitations
1616
.. under the License.
1717
18-
.. _experimental-repos:
19-
2018
Experimental repositories
2119
=========================
2220

docs/source/format/Versioning.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ the **Format Version** and the **Library Version**. Each Library
2424
Version has a corresponding Format Version, and multiple versions of
2525
the library may have the same format version. For example, library
2626
versions 2.0.0 and 3.0.0 may both track format version 1.0.0. See
27-
:doc:`status` for details about supported features in each library.
27+
:doc:`../status` for details about supported features in each library.
2828

2929
For library versions prior to 1.0.0, major releases may contain API
3030
changes. From 1.0.0 onward, we follow `Semantic Versioning

docs/source/java/cdata.rst

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ through the :ref:`c-data-interface`, even between different language runtimes.
2727
Java to Python
2828
--------------
2929

30-
Use this guide to implement :doc:`Java to Python <../python/integration/python_java.rst>`
31-
communication using the C Data Interface.
30+
See :doc:`../python/integration/python_java` to implement Java to
31+
Python communication using the C Data Interface.
3232

3333
Java to C++
3434
-----------
@@ -37,7 +37,7 @@ Example: Share an Int64 array from C++ to Java:
3737

3838
**C++ Side**
3939

40-
Use this guide to :doc:`compile arrow <../developers/cpp/building.rst>` library:
40+
See :doc:`../developers/cpp/building` to build the Arrow C++ libraries:
4141

4242
.. code-block:: shell
4343
@@ -220,4 +220,4 @@ Let's create a Java class to test our bridge:
220220
221221
C++-allocated array: [1, 2, 3, null, 5, 6, 7, 8, 9, 10]
222222
223-
.. _`JavaCPP`: https://github.com/bytedeco/javacpp
223+
.. _`JavaCPP`: https://github.com/bytedeco/javacpp

0 commit comments

Comments
 (0)