Accumulo
  1. Accumulo
  2. ACCUMULO-2174

VFS Classloader has potential to collide localized resources

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.0
    • Fix Version/s: 1.5.1, 1.6.0
    • Component/s: start
    • Labels:
      None

      Description

      Looking through the VFS implementation, a single directory is used for all Accumulo processes on a single host for a given user.

      Commons-vfs makes some attempt (adds a number to the localized resource, created from a random number between 0 and 2**16, and incremented for each updated resource) to give a "unique" resource. Rather than let this bite us later on, it's rather simple for us to just query the runtime and get the pid of the process to add into the temp directory that a process uses.

      This could increase the copies of resources per host (by the number of processes), but I think the performance penalty is much better than suddenly getting someone else's jars in the middle of my process execution.

        Activity

        Hide
        Josh Elser added a comment -

        Include process name in the temporary direction used.

        Show
        Josh Elser added a comment - Include process name in the temporary direction used.
        Hide
        ASF subversion and git services added a comment -

        Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/master from Josh Elser
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ]

        ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes

        Show
        ASF subversion and git services added a comment - Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/master from Josh Elser [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ] ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes
        Hide
        ASF subversion and git services added a comment -

        Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/1.6.0-SNAPSHOT from Josh Elser
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ]

        ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes

        Show
        ASF subversion and git services added a comment - Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/1.6.0-SNAPSHOT from Josh Elser [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ] ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes
        Hide
        ASF subversion and git services added a comment -

        Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/1.5.1-SNAPSHOT from Josh Elser
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ]

        ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes

        Show
        ASF subversion and git services added a comment - Commit db56d8d21dc636e6d271ae4c710ed27196a24506 in branch refs/heads/1.5.1-SNAPSHOT from Josh Elser [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=db56d8d ] ACCUMULO-2174 Provide a better VFS tmpdir default that won't collide across processes

          People

          • Assignee:
            Josh Elser
            Reporter:
            Josh Elser
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development