You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2012/07/02 03:45:07 UTC
Re: Review Request: Allow to download resources from any external File
Systems to local machine.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5687/#review8776
-----------------------------------------------------------
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java
<https://reviews.apache.org/r/5687/#comment18552>
Instead of regex, it might be better to use URI to parse the string.
String scheme = new Path(value).toURI().getScheme();
return (scheme != null) && !scheme.equalsIgnoreCase("file");
- Ashutosh Chauhan
On June 30, 2012, 6:15 p.m., Kanna Karanam wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5687/
> -----------------------------------------------------------
>
> (Updated June 30, 2012, 6:15 p.m.)
>
>
> Review request for hive, Carl Steinbach, Edward Capriolo, and Ashutosh Chauhan.
>
>
> Description
> -------
>
> Instead of restricting resources download to "s3", "s3n", "hdfs" make it open for any external file systems.
>
>
> This addresses bug HIVE-3146.
> https://issues.apache.org/jira/browse/HIVE-3146
>
>
> Diffs
> -----
>
> http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 1355510
> http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 1355510
>
> Diff: https://reviews.apache.org/r/5687/diff/
>
>
> Testing
> -------
>
> Yes. All unit tests passed.
>
>
> Thanks,
>
> Kanna Karanam
>
>
Re: Review Request: Allow to download resources from any external File
Systems to local machine.
Posted by Kanna Karanam <ka...@microsoft.com>.
> On July 2, 2012, 1:45 a.m., Ashutosh Chauhan wrote:
> >
Thnaks Ashutosh. It helps to understand the code easily. Please find the updated patch.
- Kanna
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5687/#review8776
-----------------------------------------------------------
On June 30, 2012, 6:15 p.m., Kanna Karanam wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5687/
> -----------------------------------------------------------
>
> (Updated June 30, 2012, 6:15 p.m.)
>
>
> Review request for hive, Carl Steinbach, Edward Capriolo, and Ashutosh Chauhan.
>
>
> Description
> -------
>
> Instead of restricting resources download to "s3", "s3n", "hdfs" make it open for any external file systems.
>
>
> This addresses bug HIVE-3146.
> https://issues.apache.org/jira/browse/HIVE-3146
>
>
> Diffs
> -----
>
> http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 1355510
> http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 1355510
>
> Diff: https://reviews.apache.org/r/5687/diff/
>
>
> Testing
> -------
>
> Yes. All unit tests passed.
>
>
> Thanks,
>
> Kanna Karanam
>
>