Uploaded image for project: 'Commons Pool'
  1. Commons Pool
  2. POOL-67

[pool] patch to remove GenericObjectPool's dep on CursorableLinkedList

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.3
    • None
    • Operating System: All
      Platform: All

    • 37428

    Description

      GenericObjectPool uses CursorableLinkedList (CLL) seemingly because the cursor/iterator from CLL allows
      concurrent access from multiple threads. Because the evictor requires exclusive access to the object pool
      the benefit of CLL is never realized. (Idle object eviction is the only place the cursor/iterator was used.)
      This patch converts the CLL into a LinkedList removing the dependancy Collections.

      This is a first step to removing any external dependancies for Pool. Currently the Collections jar is
      currently 546K compared to Pool jar which is 41K. GenericKeyedObjectPool still uses CLL and I'll convert
      that over in the near future. When I'm done with that COM-2493 will be obsolete.

      Attachments

        Issue Links

        Activity

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

          People

            Unassigned Unassigned
            sandymac@apache.org Sandy McArthur (from Bugzilla import)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment