edponce commented on a change in pull request #12044:
URL: https://github.com/apache/arrow/pull/12044#discussion_r776393469
##########
File path: csharp/src/Apache.Arrow.Flight/Server/FlightServer.cs
##########
@@ -48,9 +48,10 @@ public virtual Task DoAction(FlightAction request,
IAsyncStreamWriter<FlightResu
throw new NotImplementedException();
}
- public virtual Task<Schema> GetSchema(FlightDescriptor request,
ServerCallContext context)
+ public virtual async Task<Schema> GetSchema(FlightDescriptor request,
ServerCallContext context)
{
- throw new NotImplementedException();
+ var info = await GetFlightInfo(request, context);
+ return info.Schema;
Review comment:
This change does not seems particular to the example, so maybe resolve
in a different issue/PR. You can also include the other non-example changes
alongside the new PR.
--
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]
