[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-27 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430857565 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }

[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-26 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430857565 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }

[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-26 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430857565 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }

[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-26 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430854143 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }

[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-26 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430847155 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }

[GitHub] [arrow] kiszk commented on a change in pull request #7277: ARROW-8957: [FlightRPC][C++] directly use IpcWriteOptions

2020-05-26 Thread GitBox
kiszk commented on a change in pull request #7277: URL: https://github.com/apache/arrow/pull/7277#discussion_r430847155 ## File path: cpp/src/arrow/flight/server.cc ## @@ -352,7 +352,7 @@ class DoExchangeMessageWriter : public FlightMessageWriter { }