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

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2021/12/23 03:04:57 UTC

[GitHub] [calcite] woonsan commented on pull request #2621: [CALCITE-4908] Support classpath resource URI as model URI

woonsan commented on pull request #2621:
URL: https://github.com/apache/calcite/pull/2621#issuecomment-1000008501


   @vlsi , I guess we will probably need to take care of the potential cases where people may use file paths with spaces on different machines, but I personally prefer keeping this ticket as small as possible only focusing on `classpath:` model resource url support for now.
   Can we discuss in the dev@ list for the potential improvements regarding file paths - not classpath resource paths - with space(s) separately? It would be also great if someone can provide a real use (test) cases, too and what the real problems look like.


-- 
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: commits-unsubscribe@calcite.apache.org

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