alamb commented on code in PR #4012: URL: https://github.com/apache/arrow-rs/pull/4012#discussion_r1157727782
########## arrow-flight/src/lib.rs: ########## @@ -30,6 +30,11 @@ //! //! 2. Low level [tonic] generated [`flight_service_client`] and //! [`flight_service_server`]. +//! +//! 3. Experimental support for [Flight SQL] in [`sql`]. Requires the Review Comment: > The experimental flag allows us to "fix" such things without needing to preserve backwards compatibility Well, I guess I was thinking we break backwards compatibility fairly regularly (as in there are several API changes per release) So it isn't like I think the flight sql feature is fully ready and the API won't change -- more like I wonder if we should treat it specially from the rest of arrow-flight -- 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]
