emkornfield commented on a change in pull request #10633: URL: https://github.com/apache/arrow/pull/10633#discussion_r664725400
########## File path: go/arrow/flight/server.go ########## @@ -63,6 +104,8 @@ type server struct { // Alternatively, a grpc server can be created normally without this helper as the // grpc server generated code is still being exported. This only exists to allow // the utility of the helpers +// +// Deprecated: prefer to use NewServerWithMiddleware Review comment: should this be deprecated, if implementators don't care about middleware, is there a strong reason to use that method over this one? -- 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