Fix bug in which plasma client file descriptors were not closed. (#618)

* Fix bug in which plasma client file descriptors were not closed.

* Add logging statement when disconnecting client from plasma store.

* Fix after rebasing.

* Add more checks to plasma disconnect client.
This commit is contained in:
Robert Nishihara
2017-06-01 05:37:29 +00:00
committed by Philipp Moritz
parent bcaab78908
commit 4d51ed37b2
+5
View File
@@ -407,7 +407,12 @@ void PlasmaStore::connect_client(int listener_sock) {
}
void PlasmaStore::disconnect_client(Client *client) {
ARROW_CHECK(client != NULL);
ARROW_CHECK(client->fd > 0);
loop_->remove_file_event(client->fd);
// Close the socket.
close(client->fd);
ARROW_LOG(INFO) << "Disconnecting client on fd " << client->fd;
// If this client was using any objects, remove it from the appropriate
// lists.
for (const auto &entry : store_info_.objects) {