lidavidm commented on issue #3454: URL: https://github.com/apache/arrow-adbc/issues/3454#issuecomment-3333667548
- `cancel` is something that by definition you want to call concurrently with some other method. - To do that, you must have the struct implement `Sync`. That is a base Rust requirement, correct me if not. - But then we can call any method concurrently, and I think that might be too much freedom. (At the very least, it means that the Rust implementation has to guard against this, or we need to make all drivers guard against this - currently they generally do not.) -- 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
