zanmato1984 commented on code in PR #48742: URL: https://github.com/apache/arrow/pull/48742#discussion_r2671852837
########## cpp/src/arrow/csv/reader.cc: ########## Review Comment: Oh, don't know what's going on. Let me ask this: Given that my assumption of the cause of deadlock is correct, it doesn't seem to me that we have ownership issues: * If no error, then the main thread owns the future, which owns `self`, until all tasks are finished. * If early error, then the tasks own `self`. * In either case, `self` owns all the things: options, task group. So why do we have to make all the ownership related changes in the rest of the PR? -- 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]
