You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "James Duong (Jira)" <ji...@apache.org> on 2022/03/11 23:27:00 UTC

[jira] [Commented] (ARROW-15921) [Format][FlightRPC] Clarify FlightEndpoint.locations

    [ https://issues.apache.org/jira/browse/ARROW-15921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17505143#comment-17505143 ] 

James Duong commented on ARROW-15921:
-------------------------------------

Let's correct the examples to follow this behavior, and also follow the specified empty locations behavior (data is only at the same location as where GetFlightInfo was called):

[https://github.com/apache/arrow/blob/dc2e0b2e44fdaa3d5ad0bb358ff8ce9db3bc7416/java/flight/flight-integration-tests/src/main/java/org/apache/arrow/flight/integration/tests/IntegrationTestClient.java#L151]

[https://github.com/apache/arrow/blob/dc2e0b2e44fdaa3d5ad0bb358ff8ce9db3bc7416/cpp/src/arrow/flight/flight_benchmark.cc#L297]

Not sure if other languages need to be corrected too.

> [Format][FlightRPC] Clarify FlightEndpoint.locations
> ----------------------------------------------------
>
>                 Key: ARROW-15921
>                 URL: https://issues.apache.org/jira/browse/ARROW-15921
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: FlightRPC, Format
>            Reporter: David Li
>            Assignee: David Li
>            Priority: Major
>
> [2022/03/11 "[Java] [Flight] Usage of Locations on FlightEndpoint" on user@arrow.apache.org|https://lists.apache.org/thread/1668fk1myqf8168xh296qck5fn8ztcmn]
> The {{locations}} field of {{FlightEndpoint}} was meant to have a particular interpretation, we should make this clear in the protocol comments.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)