Hive
  1. Hive
  2. HIVE-1532

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      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.

        Activity

        He Yongqiang created issue -
        He Yongqiang made changes -
        Field Original Value New Value
        Assignee He Yongqiang [ he yongqiang ]
        He Yongqiang made changes -
        Attachment Hive-1532.1.patch [ 12452113 ]
        He Yongqiang made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Namit Jain added a comment -

        +1

        will commit if the tests pass

        Show
        Namit Jain added a comment - +1 will commit if the tests pass
        Hide
        Namit Jain added a comment -

        Committed. Thanks yongqiang

        Show
        Namit Jain added a comment - Committed. Thanks yongqiang
        Namit Jain made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Resolution Fixed [ 1 ]
        Carl Steinbach made changes -
        Fix Version/s 0.7.0 [ 12315150 ]
        Carl Steinbach made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2d 19h 2m 1 He Yongqiang 14/Aug/10 22:44
        Patch Available Patch Available Resolved Resolved
        8h 6m 1 Namit Jain 15/Aug/10 06:51
        Resolved Resolved Closed Closed
        488d 18h 8m 1 Carl Steinbach 16/Dec/11 23:59

          People

          • Assignee:
            He Yongqiang
            Reporter:
            He Yongqiang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development