joellubi commented on code in PR #38385:
URL: https://github.com/apache/arrow/pull/38385#discussion_r1387867083
##########
docs/source/format/FlightSql.rst:
##########
@@ -149,6 +149,12 @@ the ``type`` should be ``ClosePreparedStatement``).
When used with DoPut: execute the query and return the number of
affected rows.
+``CommandStatementIngest``
+ Execute a bulk ingestion.
+
+ When used with DoPut: load the stream of Arrow record batches into
+ the specified target table and return the number of rows ingested.
Review Comment:
I agree this was not made very clear. Yes there is a Flight SQL protobuf
`DoPutUpdateResult` message encoded in the `app_metadata`. I've just pushed
some updates to documentation and comments to clarify this.
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
