Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.0.0-alpha, trunk
    • Fix Version/s: None
    • Component/s: jobhistoryserver, mrv2
    • Labels:
      None

      Description

      Currently in the clean() method, we are deleting only the files/directories inside YYYY/MM/DD directory and leaving those time stamped directories (YYYY/MM/DD) as it is. As these are empty directories, I think, we should delete these time stamped directories from FileSystem in the clean method.

        Activity

        Hide
        Robert Joseph Evans added a comment -

        I saw this too, The difficult part with this is debug mode in the history server. It switches the YYYY/MM/DD to YYYY/HH/mm (or something like that). This makes it very difficult for the cleanup code to be able to know when it is allowed to delete a particular directory. It is possible, however unlikely, that the cleanup thread would try to delete a directory at the same time that someone else is trying to add something to them. This could potentially result in files being deleted that should not be. It should not be too hard to work around it, but it is something to be aware of.

        Show
        Robert Joseph Evans added a comment - I saw this too, The difficult part with this is debug mode in the history server. It switches the YYYY/MM/DD to YYYY/HH/mm (or something like that). This makes it very difficult for the cleanup code to be able to know when it is allowed to delete a particular directory. It is possible, however unlikely, that the cleanup thread would try to delete a directory at the same time that someone else is trying to add something to them. This could potentially result in files being deleted that should not be. It should not be too hard to work around it, but it is something to be aware of.

          People

          • Assignee:
            Unassigned
            Reporter:
            Bhallamudi Venkata Siva Kamesh
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:

              Development