Lucene - Core
  1. Lucene - Core
  2. LUCENE-744

TestFieldsReader - TestLazyPerformance problems w/ permissions in temp dir in multiuser environment

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Was trying to setup some enhancements to the nightly builds and the testLazyPerformance test failed in TestFieldsReader since it couldn't delete the lazyDir directory from someone else's running of the test. Will change it to append user.name System property to the directory name.

        Activity

        Hide
        Hoss Man added a comment -

        2cents: there's really no need for any of our tests to use the system temp dir ... the junit task in common-build.xml explicitly sets the system property "tempDir" as a place that tests can create files (ideally they should try to clean them up when they are done)

        Show
        Hoss Man added a comment - 2cents: there's really no need for any of our tests to use the system temp dir ... the junit task in common-build.xml explicitly sets the system property "tempDir" as a place that tests can create files (ideally they should try to clean them up when they are done)
        Hide
        Grant Ingersoll added a comment -

        Even better solution. Any reason java.io.tmpdir in JUnit task isn't
        overridden to point to the same place as tempDir?

        --------------------------
        Grant Ingersoll
        Center for Natural Language Processing
        http://www.cnlp.org

        Read the Lucene Java FAQ at http://wiki.apache.org/jakarta-lucene/
        LuceneFAQ

        Show
        Grant Ingersoll added a comment - Even better solution. Any reason java.io.tmpdir in JUnit task isn't overridden to point to the same place as tempDir? -------------------------- Grant Ingersoll Center for Natural Language Processing http://www.cnlp.org Read the Lucene Java FAQ at http://wiki.apache.org/jakarta-lucene/ LuceneFAQ
        Hide
        Grant Ingersoll added a comment -

        OK, per Chris' suggestion, I changed this to use tempDir.

        Show
        Grant Ingersoll added a comment - OK, per Chris' suggestion, I changed this to use tempDir.

          People

          • Assignee:
            Grant Ingersoll
            Reporter:
            Grant Ingersoll
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development