I meant purely the changes to the "format" directory. It would probably
be better to have those in their own PR?
Le 21/06/2023 à 15:49, David Li a écrit :
Ah, sorry, it looks like the original linked PR is not this one. There is an
implementation for C++/Java/Go at https://github.com/apache/arrow/pull/36009.
On Wed, Jun 21, 2023, at 09:49, David Li wrote:
There is a PR linked in the original message, but here it is again:
https://github.com/apache/arrow/pull/36009
Cancel and Close are close semantically, but Cancel is meant for when
the (client thinks that) computation is still ongoing, while Close is
meant to free server resources after reading a result set. (For
example, JDBC has Statement#cancel [1] and ResultSet#close [2].)
That said, I think it's reasonable to only have Cancel at the protocol level.
[1]: https://docs.oracle.com/javase/8/docs/api/java/sql/Statement.html#cancel--
[2]: https://docs.oracle.com/javase/8/docs/api/java/sql/ResultSet.html#close--
On Wed, Jun 21, 2023, at 09:35, Antoine Pitrou wrote:
Hi Kou,
Can we have an actual PR with the proposed gRPC field, method and
docstring additions?
Regardless, I have some comments and questions:
* "RefreshFlightEndpoint" suggests the server will recompute (refresh)
the results; instead I would suggest "PersistFlightEndpoint"
* Perhaps "PersistFlightEndpoint" can take an optional
"suggested_expiration" timestamp, which the server is free to ignore
(some clients may only need to extend the expiration by two minutes,
others by two days...)
* Does the client potentially have to call "PersistFlightEndpoint" on
each returned endpoint? Can it pass several endpoints at once?
* What is the expected difference between "CancelFlightInfo" and
"CloseFlightInfo"? Both seem to have a similar effect, and the exact
behaviour will probably be server-dependent anyway ("cancel" and "close"
may have meaningful differences when putting/uploading data, not so much
when getting/downloading data, IMHO?).
Regards
Antoine.
Le 21/06/2023 à 02:28, Sutou Kouhei a écrit :
Hi,
David provided the Java implementation. Thanks!
If anyone has any comments about this proposal, please share
them.
Thanks,
