raulcd commented on PR #47115: URL: https://github.com/apache/arrow/pull/47115#issuecomment-3096983775
@lidavidm I've tried to play some tricks to reuse the `ipc::RecordBatchWriter` with a new IpcPayloadWriter, similar to what we do on the client. This is the first time I am doing anything with Flight so I might have completely missed something, please let me know if that's the case :) I found a couple of special cases from our tests that surprised me because they diverge from my expectations over IPC. There's a flight sql test that seems to be changing the schema mid stream, is this expected? Can we return FlightPayloads that doesn't match the returned schema? I am not adding more tests because with the change on the Python test I am already covering the original issue for the replacement dict on DoGet and with the previous tests we cover possible regressions. Let me know if you would like other test cases to be tackled. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
