tokoko commented on code in PR #46194:
URL: https://github.com/apache/arrow/pull/46194#discussion_r2062703839


##########
docs/source/format/Flight.rst:
##########
@@ -369,6 +369,61 @@ string, so the obvious candidates are not compatible.  The 
chosen
 representation can be parsed by both implementations, as well as Go's
 ``net/url`` and Python's ``urllib.parse``.
 
+Extended Location URIs
+----------------------
+
+In addition to alternative transports, a server may also return
+URIs that reference an external service or object storage location.
+This can be useful in cases where intermediate data is cached as
+Apache Parquet files on S3 or is accessible via an HTTP service. In
+these scenarios, it is more efficient to be able to provide a URI
+where the client may simply download the data directly, rather than
+requiring a Flight service to read it back into memory and serve it
+from a ``DoGet`` request. Servers should use the following URI
+schemes for this situation:
+
++--------------------+------------------------+
+| Location           | URI Scheme             |
++====================+========================+
+| Object storage (1) | s3:, gcs:, abfs:, etc. |

Review Comment:
   fwiw, seems like delta sharing (which effectively does something very 
similar) already works like that. According to 
[spec](https://github.com/delta-io/delta-sharing/blob/main/PROTOCOL.md) urls 
for add/remove files are exclusively https, although spec claims to support S3, 
ADLS, or GCS.



-- 
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

Reply via email to