Uploaded image for project: 'Apache Storm'
  1. Apache Storm
  2. STORM-3311

Use Java 7 Files API for IO instead of the older API

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: In Progress
    • Major
    • Resolution: Unresolved
    • 2.0.0
    • None
    • None

    Description

      We should try to use the NIO Files API for file IO. The older file API causes issues on Windows, since it doesn't set the FILE_SHARE_DELETE flag when opening files. This causes Storm to be unable to delete files that have open handles, which is unlike the behavior on Linux. This can cause e.g. unnecessary supervisor crashes because one thread tries to delete a file that is open in another.

      For the same reason, we should get rid of uses of common-io FileUtils, and get rid of uses of Guava's Files class that opens IO streams.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            srdo Stig Rohde Døssing
            srdo Stig Rohde Døssing

            Dates

              Created:
              Updated:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h 50m
                1h 50m

                Slack

                  Issue deployment