arnoldwakim commented on PR #437: URL: https://github.com/apache/arrow-go/pull/437#issuecomment-3070471636
> Curious, somehow this change is causing a deadlock in the tests. I'll see if I can dig a bit and figure out the cause. I see what the issue was, and that's my bad. For some reason I did not push the code I intended in the first place. On my local machine, I ran the test when draining the channel after [this error check](https://github.com/apache/arrow-go/blob/b196d3b316d09f63786f021d4f1baa1fdd7620d2/arrow/flight/flightsql/server.go#L915), which was fine. The version I pushed was deferring before the error check causing the deadlocks since it was the equivalent of this: ```go var cc <-chan flight.StreamChunk for chunk := range cc { ... } ``` I have amended the commit and pushed the intended version. Sorry for the inconvenience. -- 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
