scruz-denodo commented on PR #43155: URL: https://github.com/apache/arrow/pull/43155#issuecomment-2216816803
So, if I understand correctly, you are suggesting to avoid the usage of the builder `org.apache.arrow.flight.FlightServer.Builder` for creating the `FlightServer` instance. Reimplementing a new builder for creating the `FlightServer` object in a controlled code where you can do whetever you want. In such case, in my opinion, it is a workaround but not the best way to do that thinking on code maintenance and future updates of the library. I think that the idea should be instantiating the `FlightServer` through the official apache-arrow builder `org.apache.arrow.flight.FlightServer.Builder`. By this way, any change on the `builder` code in future releases will not be lost for people reimplementing that logic. -- 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]
