Lucene - Core
  1. Lucene - Core
  2. LUCENE-6637

FSTTester writes to CWD if -Dtests.verbose failing with AccessControlException(FilePermission)

    Details

    • Lucene Fields:
      New

      Description

      This was found by Mesbah (M.) Alam who is testing our stuff with IBM J9. He ran tests with -Dtests.verbose=true, hitting this bug in FSTTester. I would like to remove the whole toDot() stuff there - or pass it to System.out instead. Another alternative would be to write it to FST's working dir (dir.createOutput; but unfortunately this is no OutputStream or Writer...)

      Any opinions?

      1. LUCENE-6637.patch
        2 kB
        Uwe Schindler
      2. LUCENE-6637.patch
        1 kB
        Uwe Schindler

        Activity

        Hide
        Uwe Schindler added a comment -

        This is my quick fix for it (it prints the .dot file to System.out)

        Show
        Uwe Schindler added a comment - This is my quick fix for it (it prints the .dot file to System.out)
        Hide
        Michael McCandless added a comment -

        Woops, sorry about that This was very helpful while getting this test working!

        +1 to the patch, System.out is fine. Thanks Uwe Schindler

        Show
        Michael McCandless added a comment - Woops, sorry about that This was very helpful while getting this test working! +1 to the patch, System.out is fine. Thanks Uwe Schindler
        Hide
        Uwe Schindler added a comment -

        new patch without BufferedOutputStream (useless). Will commit now.

        Show
        Uwe Schindler added a comment - new patch without BufferedOutputStream (useless). Will commit now.
        Hide
        ASF subversion and git services added a comment -

        Commit 1688317 from Uwe Schindler in branch 'dev/trunk'
        [ https://svn.apache.org/r1688317 ]

        LUCENE-6637: Fix FSTTester to not violate file permissions on -Dtests.verbose=true

        Show
        ASF subversion and git services added a comment - Commit 1688317 from Uwe Schindler in branch 'dev/trunk' [ https://svn.apache.org/r1688317 ] LUCENE-6637 : Fix FSTTester to not violate file permissions on -Dtests.verbose=true
        Hide
        ASF subversion and git services added a comment -

        Commit 1688318 from Uwe Schindler in branch 'dev/branches/branch_5x'
        [ https://svn.apache.org/r1688318 ]

        Merged revision(s) 1688317 from lucene/dev/trunk:
        LUCENE-6637: Fix FSTTester to not violate file permissions on -Dtests.verbose=true

        Show
        ASF subversion and git services added a comment - Commit 1688318 from Uwe Schindler in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1688318 ] Merged revision(s) 1688317 from lucene/dev/trunk: LUCENE-6637 : Fix FSTTester to not violate file permissions on -Dtests.verbose=true
        Hide
        Uwe Schindler added a comment - - edited

        Thanks Mesbah Alam!

        Show
        Uwe Schindler added a comment - - edited Thanks Mesbah Alam!
        Hide
        Shalin Shekhar Mangar added a comment -

        Bulk close for 5.3.0 release

        Show
        Shalin Shekhar Mangar added a comment - Bulk close for 5.3.0 release

          People

          • Assignee:
            Uwe Schindler
            Reporter:
            Uwe Schindler
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development