AntiO2 commented on code in PR #2287: URL: https://github.com/apache/datafusion-ballista/pull/2287#discussion_r3764702258
########## docs/source/contributors-guide/architecture.md: ########## @@ -72,7 +72,7 @@ between the executor(s) and the scheduler for fetching tasks and reporting task The scheduler provides the following interfaces: - gRPC service for submitting and managing jobs -- Flight SQL API +- optional Arrow Flight proxy for fetching query results Review Comment: Agreed, removed this entry. ########## docs/source/contributors-guide/code-organization.md: ########## @@ -33,7 +33,7 @@ This section provides links to the source code for major areas of functionality. - [Crate Source](https://github.com/apache/datafusion-ballista/tree/main/ballista/scheduler) - [Distributed Query Planner](https://github.com/apache/datafusion-ballista/blob/main/ballista/scheduler/src/planner.rs) - [gRPC Service](https://github.com/apache/datafusion-ballista/blob/main/ballista/scheduler/src/scheduler_server/grpc.rs) -- [Flight SQL Service](https://github.com/apache/datafusion-ballista/blob/main/ballista/scheduler/src/flight_sql.rs) +- [Flight Proxy Service](https://github.com/apache/datafusion-ballista/blob/main/ballista/scheduler/src/flight_proxy_service.rs) Review Comment: Agreed, removed this entry. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
