You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "He Yongqiang (JIRA)" <ji...@apache.org> on 2010/08/12 04:44:15 UTC

[jira] Created: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Replace globStatus with listStatus inside Hive.java's replaceFiles.
-------------------------------------------------------------------

                 Key: HIVE-1532
                 URL: https://issues.apache.org/jira/browse/HIVE-1532
             Project: Hadoop Hive
          Issue Type: Bug
            Reporter: He Yongqiang


globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.

We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Posted by "He Yongqiang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-1532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

He Yongqiang updated HIVE-1532:
-------------------------------

    Attachment: Hive-1532.1.patch

> Replace globStatus with listStatus inside Hive.java's replaceFiles.
> -------------------------------------------------------------------
>
>                 Key: HIVE-1532
>                 URL: https://issues.apache.org/jira/browse/HIVE-1532
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: He Yongqiang
>            Assignee: He Yongqiang
>         Attachments: Hive-1532.1.patch
>
>
> globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.
> We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Posted by "He Yongqiang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-1532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

He Yongqiang reassigned HIVE-1532:
----------------------------------

    Assignee: He Yongqiang

> Replace globStatus with listStatus inside Hive.java's replaceFiles.
> -------------------------------------------------------------------
>
>                 Key: HIVE-1532
>                 URL: https://issues.apache.org/jira/browse/HIVE-1532
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: He Yongqiang
>            Assignee: He Yongqiang
>
> globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.
> We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Posted by "Namit Jain (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-1532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Namit Jain updated HIVE-1532:
-----------------------------

          Status: Resolved  (was: Patch Available)
    Hadoop Flags: [Reviewed]
      Resolution: Fixed

Committed. Thanks yongqiang

> Replace globStatus with listStatus inside Hive.java's replaceFiles.
> -------------------------------------------------------------------
>
>                 Key: HIVE-1532
>                 URL: https://issues.apache.org/jira/browse/HIVE-1532
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: He Yongqiang
>            Assignee: He Yongqiang
>         Attachments: Hive-1532.1.patch
>
>
> globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.
> We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Posted by "He Yongqiang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HIVE-1532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

He Yongqiang updated HIVE-1532:
-------------------------------

    Status: Patch Available  (was: Open)

> Replace globStatus with listStatus inside Hive.java's replaceFiles.
> -------------------------------------------------------------------
>
>                 Key: HIVE-1532
>                 URL: https://issues.apache.org/jira/browse/HIVE-1532
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: He Yongqiang
>            Assignee: He Yongqiang
>         Attachments: Hive-1532.1.patch
>
>
> globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.
> We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HIVE-1532) Replace globStatus with listStatus inside Hive.java's replaceFiles.

Posted by "Namit Jain (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HIVE-1532?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12898653#action_12898653 ] 

Namit Jain commented on HIVE-1532:
----------------------------------

+1

will commit if the tests pass

> Replace globStatus with listStatus inside Hive.java's replaceFiles.
> -------------------------------------------------------------------
>
>                 Key: HIVE-1532
>                 URL: https://issues.apache.org/jira/browse/HIVE-1532
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: He Yongqiang
>            Assignee: He Yongqiang
>         Attachments: Hive-1532.1.patch
>
>
> globStatus expects a regular expression,  so if there is special characters (like '{' , '[') in the filepath, this function will fail.
> We should be able to replace this call with listStatus easily since we are not passing regex to replaceFiles(). The only places replaceFiles is called is in loadPartition and Table's replaceFiles.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.