[GitHub] [arrow] vibhatha commented on a diff in pull request #33725: GH-33724: [Doc] Update the substrait conformance doc with the latest support

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

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org