You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/03/21 16:51:00 UTC

[GitHub] [arrow] andygrove commented on pull request #9762: ARROW-12037: [Rust] [DataFusion] Support catalogs and schemas for table namespacing

andygrove commented on pull request #9762:
URL: https://github.com/apache/arrow/pull/9762#issuecomment-803620665


   Thanks @returnString and I appreciate the well-written design doc to explain the PR. I didn't go through the code in great detail but I like the design. I noted a couple of unwraps in the code. It would be good to document why they are safe or consider having those methods return `Result` if they are not safe.


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org