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 (#3).
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. This increases
the number of
opened Kud clients by a MR application at most by one (The one that was shared
by getSplits()
with a TableRecordReader)
Also clarified the behaviour of MR applications and how many open Kudu clients
one might have to expect in total.
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/3
--
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: 3
Gerrit-Owner: Clemens Valiente <clemens.valie...@gmail.com>
Gerrit-Reviewer: Clemens Valiente <clemens.valie...@gmail.com>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
