You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Clemens Valiente (Code Review)" <ge...@cloudera.org> on 2018/01/04 09:38:51 UTC

[kudu-CR] KUDU-2249 Prevent race condition between getSplits() method and TableRecordReader

Hello David Ribeiro Alves, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: KUDU-2249 Prevent race condition between getSplits() method and TableRecordReader
......................................................................

KUDU-2249 Prevent race condition between getSplits() method and TableRecordReader

Both are sharing the same client and shut it down after use. In some scenarios the client might
still be accessed after that and throwing an error.
So the TableRecordReader gets its own client with this commit.
Also clarified the behaviour of MR applications and how many open Kudu clients
one might have to expect.

Change-Id: I24f45ee9253790c5348cabd0afe6c6a4b6d3f3d4
---
M java/kudu-mapreduce/src/main/java/org/apache/kudu/mapreduce/KuduTableInputFormat.java
1 file changed, 45 insertions(+), 21 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/21/8921/2
-- 
To view, visit http://gerrit.cloudera.org:8080/8921
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I24f45ee9253790c5348cabd0afe6c6a4b6d3f3d4
Gerrit-Change-Number: 8921
Gerrit-PatchSet: 2
Gerrit-Owner: Clemens Valiente <cl...@gmail.com>
Gerrit-Reviewer: Clemens Valiente <cl...@gmail.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins