ActiveMQ
  1. ActiveMQ
  2. AMQ-1883

KahaStore is unable to create lock file when user.dir is manually changed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 5.1.0
    • Fix Version/s: NEEDS_REVIEW
    • Component/s: Message Store
    • Labels:
      None
    • Environment:

      WinNTFileSystem
      jdk 1.6_05

      Description

      I've been getting the following exception

      Caused by: java.io.FileNotFoundException: activemq-data\localhost-kahastore\lock (The system cannot find the path specified)
      at java.io.RandomAccessFile.open(Native Method)
      at java.io.RandomAccessFile.<init>(RandomAccessFile.java:212)
      at org.apache.activemq.kaha.impl.KahaStore.initialize(KahaStore.java:476)

      I assume that changeset 661435 <http://fisheye6.atlassian.com/changelog/activemq?cs=661435> was aimed to fix the problem following problem:
      Error in directory.mkdirs is masked and can result in FileNotFoundException when creating tmp_storage directory <http://open.iona.com/issues/browse/MB-360>

      I verified that there is no problem in creating the parent directories, but filesystem thinks that directory doesn't exists after it has been created succesfully. After little more digging I stumbled upon the following:
      File.getAbsolutePath() is incorrect if "user.dir" is set on the command line <http://bugs.sun.com/bugdatabase/view_bug.do;:YfiG?bug_id=4117557>

      There is some more information about using canonical form at Mule project <http://mule.mulesource.org/jira/browse/MULE-1112> and I found out that using getCanonicalFile() does indeed work.

      As a patch I suggest that all file/directory related methods at IOHelper would return (canonical) File instead of String.

      Other not so intrusive possibility would be to change XBeanBrokerService.
      KahaStore.<init>(File, String, AtomicLong) line: 103
      StoreFactory.open(File, String) line: 56
      XBeanBrokerService(BrokerService).getTempDataStore() line: 1178

        Activity

        Timothy Bish made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Won't Fix [ 2 ]
        Jeff Turner made changes -
        Project Import Fri Nov 26 22:32:02 EST 2010 [ 1290828722158 ]
        Rob Davies made changes -
        Fix Version/s 5.4.0 [ 12110 ]
        Fix Version/s NEEDS_REVIEWED [ 12186 ]
        Gary Tully made changes -
        Fix Version/s 5.4.0 [ 12110 ]
        Fix Version/s 5.3.0 [ 11914 ]
        Rob Davies made changes -
        Assignee Rob Davies [ rajdavies ]
        Rob Davies made changes -
        Fix Version/s 5.3.0 [ 11914 ]
        Tuomas Kiviaho made changes -
        Field Original Value New Value
        Link This issue is related to AMQ-1223 [ AMQ-1223 ]
        Tuomas Kiviaho created issue -

          People

          • Assignee:
            Rob Davies
            Reporter:
            Tuomas Kiviaho
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development