FAQ
[ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated HADOOP-771:
---------------------------------

Attachment: Hadoop-771_3.patch

this patch fixes dhruba's comments.
Namenode should return error when trying to delete non-empty directory
----------------------------------------------------------------------

Key: HADOOP-771
URL: https://issues.apache.org/jira/browse/HADOOP-771
Project: Hadoop Core
Issue Type: Improvement
Components: dfs
Affects Versions: 0.8.0
Environment: all
Reporter: Milind Bhandarkar
Assignee: Mahadev konar
Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, patch_771_1.patch


Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Search Discussions

  • Mahadev konar (JIRA) at Mar 5, 2008 at 7:45 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Attachment: Hadoop-771_4.patch

    uploading a new patch with dhruba's comments incorporated.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 5, 2008 at 7:46 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Fix Version/s: 0.17.0
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 5, 2008 at 7:46 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 6, 2008 at 8:54 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 6, 2008 at 8:54 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)

    retrying hudson.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 6, 2008 at 11:42 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)

    forgot to remove delete(path) uses!

    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 2:56 am
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Attachment: Hadoop-771_5.patch

    updated patch getting rid of the javac warnings -- replacing all the uses of delete(path) with delete(path, true).
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 2:58 am
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 7:29 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 7:31 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)

    trying hudson!!
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 8:49 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Attachment: Hadoop-771_6.patch

    attaching a new patch removing some more javac warnings.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 8:49 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 7, 2008 at 8:51 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 5:53 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 5:53 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)

    trying hudson again....
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:11 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:11 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:11 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Attachment: Hadoop-771_7.patch

    uploading the patch that applies to trunk.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:25 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Open (was: Patch Available)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:29 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Attachment: Hadoop-771_8.patch

    the last patch had a conflict... resolving and uploading a new one.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, Hadoop-771_8.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Mahadev konar (JIRA) at Mar 9, 2008 at 6:29 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Mahadev konar updated HADOOP-771:
    ---------------------------------

    Status: Patch Available (was: Open)
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, Hadoop-771_8.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • dhruba borthakur (JIRA) at Mar 10, 2008 at 7:00 pm
    [ https://issues.apache.org/jira/browse/HADOOP-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    dhruba borthakur updated HADOOP-771:
    ------------------------------------

    Resolution: Fixed
    Status: Resolved (was: Patch Available)

    I just committed this. Thanks Mahadev.
    Namenode should return error when trying to delete non-empty directory
    ----------------------------------------------------------------------

    Key: HADOOP-771
    URL: https://issues.apache.org/jira/browse/HADOOP-771
    Project: Hadoop Core
    Issue Type: Improvement
    Components: dfs
    Affects Versions: 0.8.0
    Environment: all
    Reporter: Milind Bhandarkar
    Assignee: Mahadev konar
    Fix For: 0.17.0

    Attachments: Hadoop-771_2.patch, Hadoop-771_3.patch, Hadoop-771_4.patch, Hadoop-771_5.patch, Hadoop-771_6.patch, Hadoop-771_7.patch, Hadoop-771_8.patch, patch_771_1.patch


    Currently, the namenode.delete() method allows recursive deletion of a directory. That is, even a non-empty directory could be deleted using namenode.delete(). To avoid costly programmer errors, the namenode should not remove the non-empty directories in this method. Recursively deleting directory should either be performed with listPaths() followed by a delete() for every path, or with a specific namenode method such as deleteRecursive().
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupcommon-dev @
categorieshadoop
postedMar 4, '08 at 1:38a
activeMar 10, '08 at 7:00p
posts23
users1
websitehadoop.apache.org...
irc#hadoop

1 user in discussion

dhruba borthakur (JIRA): 23 posts

People

Translate

site design / logo © 2022 Grokbase