You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Mike Percy (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/01/06 05:26:17 UTC

[kudu-CR] KUDU-1288. Release WAL file descriptors when deleting tablets

Mike Percy has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/1715

Change subject: KUDU-1288. Release WAL file descriptors when deleting tablets
......................................................................

KUDU-1288. Release WAL file descriptors when deleting tablets

This is a bug fix for an issue where we would leak WAL file FDs when
deleting tablets. Over time, this would cause Kudu to run out of file
descriptors and crash.

Added Close() methods to several of the objects in the log subsystem so
that we wouldn't be at the mercy of destructors of refcounted objects.

Change-Id: I97dfac521bcd61d2b286c9a437310f40cbe7b288
---
M src/kudu/consensus/log.cc
M src/kudu/consensus/log_index.cc
M src/kudu/consensus/log_index.h
M src/kudu/consensus/log_reader.cc
M src/kudu/consensus/log_reader.h
M src/kudu/integration-tests/delete_table-test.cc
6 files changed, 135 insertions(+), 14 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/1715/1
-- 
To view, visit http://gerrit.cloudera.org:8080/1715
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I97dfac521bcd61d2b286c9a437310f40cbe7b288
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mp...@cloudera.com>