Skip to content

Commit

Permalink
Fix memory leaking when loading data: Alluxio#17758
Browse files Browse the repository at this point in the history
  • Loading branch information
humengyu2012 committed Jul 13, 2023
1 parent cfdf617 commit 9d27cf6
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -339,12 +339,12 @@ public CompletableFuture<List<BlockStatus>> load(List<Block> blocks, UfsReadOpti
blockWriter.close();
} catch (IOException e) {
throw AlluxioRuntimeException.from(e);
} finally {
NioDirectBufferPool.release(buf);
}
})
.thenRun(() -> commitBlock(sessionId, blockId, false))
.thenRun(() -> NioDirectBufferPool.release(buf))
.exceptionally(t -> {
NioDirectBufferPool.release(buf);
handleException(t.getCause(), block, errors, sessionId);
return null;
});
Expand Down
13 changes: 13 additions & 0 deletions joker.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
version: 2.0
base_image: bionic
components:
java: 1.8

build:
- echo 0

test:
unittest:
- echo 0


0 comments on commit 9d27cf6

Please sign in to comment.