You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ap...@apache.org on 2022/06/14 18:21:32 UTC
[arrow] branch master updated: ARROW-16789: [Format] Remove experimental marker from C Streaming Interface (#13345)
This is an automated email from the ASF dual-hosted git repository.
apitrou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new 9cbb8a1a62 ARROW-16789: [Format] Remove experimental marker from C Streaming Interface (#13345)
9cbb8a1a62 is described below
commit 9cbb8a1a626ee301cfe85905b6c18c5d880e176b
Author: Will Jones <wi...@gmail.com>
AuthorDate: Tue Jun 14 20:21:26 2022 +0200
ARROW-16789: [Format] Remove experimental marker from C Streaming Interface (#13345)
Approved by the PMC: https://lists.apache.org/thread/39wsd1y8f7fk1k44krm5jrw3pv6dnm45
Lead-authored-by: Will Jones <wi...@gmail.com>
Co-authored-by: Antoine Pitrou <pi...@free.fr>
Signed-off-by: Antoine Pitrou <pi...@free.fr>
---
cpp/src/arrow/c/abi.h | 2 --
cpp/src/arrow/c/bridge.h | 4 +--
docs/source/format/CStreamInterface.rst | 5 ----
docs/source/status.rst | 44 ++++++++++++++++-----------------
go/arrow/cdata/arrow/c/abi.h | 2 --
go/arrow/cdata/cdata.go | 3 +--
java/c/src/main/cpp/abi.h | 2 --
python/pyarrow/ipc.pxi | 2 +-
8 files changed, 26 insertions(+), 38 deletions(-)
diff --git a/cpp/src/arrow/c/abi.h b/cpp/src/arrow/c/abi.h
index aa13cae35b..d58417e6fb 100644
--- a/cpp/src/arrow/c/abi.h
+++ b/cpp/src/arrow/c/abi.h
@@ -65,8 +65,6 @@ struct ArrowArray {
#endif // ARROW_C_DATA_INTERFACE
-// EXPERIMENTAL: C stream interface
-
#ifndef ARROW_C_STREAM_INTERFACE
#define ARROW_C_STREAM_INTERFACE
diff --git a/cpp/src/arrow/c/bridge.h b/cpp/src/arrow/c/bridge.h
index 294f53e49f..3b1a013d20 100644
--- a/cpp/src/arrow/c/bridge.h
+++ b/cpp/src/arrow/c/bridge.h
@@ -170,7 +170,7 @@ Result<std::shared_ptr<RecordBatch>> ImportRecordBatch(struct ArrowArray* array,
///
/// @{
-/// \brief EXPERIMENTAL: Export C++ RecordBatchReader using the C stream interface.
+/// \brief Export C++ RecordBatchReader using the C stream interface.
///
/// The resulting ArrowArrayStream struct keeps the record batch reader alive
/// until its release callback is called by the consumer.
@@ -181,7 +181,7 @@ ARROW_EXPORT
Status ExportRecordBatchReader(std::shared_ptr<RecordBatchReader> reader,
struct ArrowArrayStream* out);
-/// \brief EXPERIMENTAL: Import C++ RecordBatchReader from the C stream interface.
+/// \brief Import C++ RecordBatchReader from the C stream interface.
///
/// The ArrowArrayStream struct has its contents moved to a private object
/// held alive by the resulting record batch reader.
diff --git a/docs/source/format/CStreamInterface.rst b/docs/source/format/CStreamInterface.rst
index a35f2045c9..7d61be23a0 100644
--- a/docs/source/format/CStreamInterface.rst
+++ b/docs/source/format/CStreamInterface.rst
@@ -23,11 +23,6 @@
The Arrow C stream interface
============================
-.. warning::
- This interface is experimental and may evolve based on feedback from
- early users. ABI stability is not guaranteed yet. Feel free to
- `contact us <https://arrow.apache.org/community/>`__.
-
The C stream interface builds on the structures defined in the
:ref:`C data interface <c-data-interface>` and combines them into a higher-level
specification so as to ease the communication of streaming data within a single
diff --git a/docs/source/status.rst b/docs/source/status.rst
index 95f018980b..4caf0eb156 100644
--- a/docs/source/status.rst
+++ b/docs/source/status.rst
@@ -224,34 +224,34 @@ Notes:
C Data Interface
================
-+-----------------------------+-----+--------+---+------+----+------+--------+------+
-| Feature | C++ | Python | R | Rust | Go | Java | C/GLib | Ruby |
-| | | | | | | | | |
-+=============================+=====+========+===+======+====+======+========+======+
-| Schema export | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ |
-+-----------------------------+-----+--------+---+------+----+------+--------+------+
-| Array export | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ |
-+-----------------------------+-----+--------+---+------+----+------+--------+------+
-| Schema import | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ |
-+-----------------------------+-----+--------+---+------+----+------+--------+------+
-| Array import | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ |
-+-----------------------------+-----+--------+---+------+----+------+--------+------+
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Feature | C++ | Python | R | Rust | Go | Java | C/GLib | Ruby | Julia |
+| | | | | | | | | | |
++=============================+=====+========+===+======+====+======+========+======+=======+
+| Schema export | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Array export | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Schema import | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Array import | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
.. seealso::
The :ref:`C Data Interface <c-data-interface>` specification.
-C Stream Interface (experimental)
-=================================
+C Stream Interface
+==================
-+-----------------------------+-----+--------+----+--------+------+
-| Feature | C++ | Python | Go | C/GLib | Ruby |
-| | | | | | |
-+=============================+=====+========+====+========+======+
-| Stream export | ✓ | ✓ | | ✓ | ✓ |
-+-----------------------------+-----+--------+----+--------+------+
-| Stream import | ✓ | ✓ | ✓ | ✓ | ✓ |
-+-----------------------------+-----+--------+----+--------+------+
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Feature | C++ | Python | R | Rust | Go | Java | C/GLib | Ruby | Julia |
+| | | | | | | | | | |
++=============================+=====+========+===+======+====+======+========+======+=======+
+| Stream export | ✓ | ✓ | ✓ | ✓ | | | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
+| Stream import | ✓ | ✓ | ✓ | ✓ | ✓ | | ✓ | ✓ | |
++-----------------------------+-----+--------+---+------+----+------+--------+------+-------+
.. seealso::
The :ref:`C Stream Interface <c-stream-interface>` specification.
diff --git a/go/arrow/cdata/arrow/c/abi.h b/go/arrow/cdata/arrow/c/abi.h
index aa13cae35b..d58417e6fb 100644
--- a/go/arrow/cdata/arrow/c/abi.h
+++ b/go/arrow/cdata/arrow/c/abi.h
@@ -65,8 +65,6 @@ struct ArrowArray {
#endif // ARROW_C_DATA_INTERFACE
-// EXPERIMENTAL: C stream interface
-
#ifndef ARROW_C_STREAM_INTERFACE
#define ARROW_C_STREAM_INTERFACE
diff --git a/go/arrow/cdata/cdata.go b/go/arrow/cdata/cdata.go
index 7436c10ebf..5af7ddb7d2 100644
--- a/go/arrow/cdata/cdata.go
+++ b/go/arrow/cdata/cdata.go
@@ -54,8 +54,7 @@ type (
CArrowSchema = C.struct_ArrowSchema
// CArrowArray is the C Data Interface object for Arrow Arrays as defined in abi.h
CArrowArray = C.struct_ArrowArray
- // CArrowArrayStream is the Experimental API for handling streams of record batches
- // through the C Data interface.
+ // CArrowArrayStream is the C Stream Interface object for handling streams of record batches.
CArrowArrayStream = C.struct_ArrowArrayStream
)
diff --git a/java/c/src/main/cpp/abi.h b/java/c/src/main/cpp/abi.h
index aa13cae35b..d58417e6fb 100644
--- a/java/c/src/main/cpp/abi.h
+++ b/java/c/src/main/cpp/abi.h
@@ -65,8 +65,6 @@ struct ArrowArray {
#endif // ARROW_C_DATA_INTERFACE
-// EXPERIMENTAL: C stream interface
-
#ifndef ARROW_C_STREAM_INTERFACE
#define ARROW_C_STREAM_INTERFACE
diff --git a/python/pyarrow/ipc.pxi b/python/pyarrow/ipc.pxi
index d6ec976655..8b3784fae5 100644
--- a/python/pyarrow/ipc.pxi
+++ b/python/pyarrow/ipc.pxi
@@ -609,7 +609,7 @@ cdef class RecordBatchReader(_Weakrefable):
-----
To import and export using the Arrow C stream interface, use the
``_import_from_c`` and ``_export_from_c`` methods. However, keep in mind this
- interface is experimental and intended for expert users.
+ interface is intended for expert users.
Examples
--------