tustvold commented on code in PR #4012:
URL: https://github.com/apache/arrow-rs/pull/4012#discussion_r1157710824


##########
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:
   Where did we get to with an integration test of FlightSQL? I think that 
would help justify graduating this functionality perhaps? 
   
   I'm somewhat apprehensive there may still be areas we have interpreted the 
specification differently, one was fixed last week. The experimental flag 
allows us to "fix" such things without needing to preserve backwards 
compatibility



-- 
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]

Reply via email to