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. Text File
            LUCENE-6637.patch
            2 kB
            Uwe Schindler
          2. Text File
            LUCENE-6637.patch
            1 kB
            Uwe Schindler

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            Uwe Schindler added a comment - - edited

            Thanks Mesbah Alam!

            Show
            Uwe Schindler added a comment - - edited Thanks Mesbah Alam!
            Hide
            Permalink
            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