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:28:13 UTC

[kudu-CR] KUDU-2249 Prevent a possible race conditions between the getSplits() method and TableRecordReader class

Clemens Valiente has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8937


Change subject: KUDU-2249 Prevent a possible race conditions between the getSplits() method and TableRecordReader class
......................................................................

KUDU-2249 Prevent a possible race conditions between the getSplits() method and TableRecordReader class

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. Also clarified the behaviour of MR applications and how many
open Kudu clients one might have to expect.

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



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6508a02edaba76f3234e64e2a44b4846dc7e818b
Gerrit-Change-Number: 8937
Gerrit-PatchSet: 1
Gerrit-Owner: Clemens Valiente <cl...@gmail.com>

[kudu-CR] KUDU-2249 Prevent a possible race conditions between the getSplits() method and TableRecordReader class

Posted by "Clemens Valiente (Code Review)" <ge...@cloudera.org>.
Clemens Valiente has abandoned this change. ( http://gerrit.cloudera.org:8080/8937 )

Change subject: KUDU-2249 Prevent a possible race conditions between the getSplits() method and TableRecordReader class
......................................................................


Abandoned
-- 
To view, visit http://gerrit.cloudera.org:8080/8937
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: abandon
Gerrit-Change-Id: I6508a02edaba76f3234e64e2a44b4846dc7e818b
Gerrit-Change-Number: 8937
Gerrit-PatchSet: 1
Gerrit-Owner: Clemens Valiente <cl...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins