alamb commented on PR #7253: URL: https://github.com/apache/arrow-rs/pull/7253#issuecomment-2719252628
I will try and review this a bit more carefully tomorrow I think it could do with some more documentation / examples (which I can help with) I also think the name `SpawnService` while it makes sense given the underlying implementation (to spawn on another runtime) is somewhat confusing. I recommend a more explicit name like `CrossRTService` or `CrossRTHttpService` -- I realize I may prefer longer names than @tustvold , but given this one would likely be used only a few times (when setting up the object store) I think being more explicit is better -- 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
