vibhatha commented on code in PR #33725:
URL: https://github.com/apache/arrow/pull/33725#discussion_r1072460575
##########
docs/source/cpp/streaming_execution.rst:
##########
@@ -86,25 +86,21 @@ Plans
^^^^^
* A plan should have a single top-level relation.
- * The consumer is currently based on a custom build of Substrait that
- is older than 0.1.0. Any features added that are newer than 0.1.0 will
- not be supported.
+ * The consumer is currently based on a version 0.20.0 of Substrait.
+ Any features added that are newer will not be supported.
+ * Due to a breaking change in 0.20.0 any Substrait plan older than 0.20.0
Review Comment:
Do we have a validation mechanism for this?
##########
docs/source/cpp/streaming_execution.rst:
##########
@@ -113,12 +109,12 @@ Read Relations
* The ``projection`` property is not supported and plans containing this
property will be rejected.
- * The only supported read type is ``LocalFiles``. Plans with any other
- type will be rejected.
- * Only the parquet file format is currently supported.
+ * The only ``VirtualTable`` and ``ExtensionTable``read types are not
supported.
Review Comment:
@westonpace should we add unsupported status as a warning for plans with
Virtual Table?
--
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]
