Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Component/s: Utilities
    • Labels:
      None

      Description

      IOUtils.closeQuietly(Selector) would be useful to fix yet another resource leakage.

        Activity

        Karthik K created issue -
        Karthik K made changes -
        Field Original Value New Value
        Summary commons-io: IOUtils.closeQuietly(Selector) added commons-io: IOUtils.closeQuietly(Selector) necessary
        Description IOUtils.closeQuietly(Selector) added in place. IOUtils.closeQuietly(Selector) would be useful to fix yet another resource leakage.
        Karthik K made changes -
        Attachment IO-301.patch [ 12511228 ]
        Karthik K made changes -
        Attachment IO-301.patch [ 12511228 ]
        Karthik K made changes -
        Attachment IO-301.patch [ 12511229 ]
        Hide
        Karthik K added a comment -

        IOUtils.closeQuietly(Selector) has some test cases as well.

        Show
        Karthik K added a comment - IOUtils.closeQuietly(Selector) has some test cases as well.
        Gary Gregory made changes -
        Summary commons-io: IOUtils.closeQuietly(Selector) necessary Add IOUtils.closeQuietly(Selector)
        Hide
        Gary Gregory added a comment -

        Patch applied, thank you. Also added a test to get test code coverage to 100% for the new method.

        Show
        Gary Gregory added a comment - Patch applied, thank you. Also added a test to get test code coverage to 100% for the new method.
        Gary Gregory made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Gary Gregory added a comment -

        Version 2.2 has been released and addresses this issue.

        Show
        Gary Gregory added a comment - Version 2.2 has been released and addresses this issue.
        Gary Gregory made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        9h 29m 1 Gary Gregory 20/Jan/12 16:09
        Resolved Resolved Closed Closed
        69d 5h 22m 1 Gary Gregory 29/Mar/12 21:31

          People

          • Assignee:
            Unassigned
            Reporter:
            Karthik K
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development