opensourcegeek commented on code in PR #5728:
URL: https://github.com/apache/arrow-rs/pull/5728#discussion_r1592140812


##########
arrow-flight/src/client.rs:
##########
@@ -704,3 +679,75 @@ impl FlightClient {
         request
     }
 }
+
+struct FallibleRequestStream {

Review Comment:
   I did refactor it so that this is generic and not tied to request stream 
itself, is it worth moving this bit of code to some internal_utils module? 
Happy to leave it where it is atm



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