[ 
https://issues.apache.org/jira/browse/HADOOP-18105?focusedWorklogId=771801&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-771801
 ]

ASF GitHub Bot logged work on HADOOP-18105:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 18/May/22 11:07
            Start Date: 18/May/22 11:07
    Worklog Time Spent: 10m 
      Work Description: steveloughran commented on code in PR #4263:
URL: https://github.com/apache/hadoop/pull/4263#discussion_r875762848


##########
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestMoreWeakReferencedElasticByteBufferPool.java:
##########
@@ -52,6 +55,32 @@ public void testMixedBuffersInPool() {
 
   }
 
+  @Test
+  public void testUnexpectedBufferSizes() {
+    WeakReferencedElasticByteBufferPool pool = new 
WeakReferencedElasticByteBufferPool();
+    ByteBuffer buffer1 = pool.getBuffer(true, 0);
+    try {

Review Comment:
   use LambdaTestUtils.intercept() here and below





Issue Time Tracking
-------------------

    Worklog Id:     (was: 771801)
    Time Spent: 2h 10m  (was: 2h)

> Implement a variant of ElasticByteBufferPool which uses weak references for 
> garbage collection.
> -----------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-18105
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18105
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: common, fs
>            Reporter: Mukund Thakur
>            Assignee: Mukund Thakur
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 2h 10m
>  Remaining Estimate: 0h
>
> Currently in hadoop codebase, we have two classes which implements byte 
> buffers pooling.
> One is ElasticByteBufferPool which doesn't use weak references and thus could 
> cause memory leaks in production environment. 
> Other is DirectBufferPool which uses weak references but doesn't support 
> caller's preference for either on-heap or off-heap buffers. 
>  
> The idea is to create an improved version of ElasticByteBufferPool by 
> subclassing it ( as it is marked as public and stable and used widely in hdfs 
> ) with essential functionalities required for effective buffer pooling. This 
> is important for the parent Vectored IO work.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to