[kudu-CR] [java client] Fix GetTableLocations error handling
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3295 to look at the new patch set (#4). Change subject: [java client] Fix GetTableLocations error handling .. [java client] Fix GetTableLocations error handling When GetTableLocations get an error, e.g. table not found, in some scenario the callback will silently drop the error, not notifying the RPC which triggered this GetTableLocations call, causing the RPC to hang. This usually happens when user is inserting data, but underlying table is deleted and recreated. Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 --- M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java A java/kudu-client/src/main/java/org/kududb/client/ListTabletsRequest.java A java/kudu-client/src/main/java/org/kududb/client/ListTabletsResponse.java M java/kudu-client/src/test/java/org/kududb/client/TestAsyncKuduSession.java 4 files changed, 157 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/3295/4 -- To view, visit http://gerrit.cloudera.org:8080/3295 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Binglin ChangGerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] [java client] Fix GetTableLocations error handling
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3295 to look at the new patch set (#3). Change subject: [java client] Fix GetTableLocations error handling .. [java client] Fix GetTableLocations error handling When GetTableLocations get an error, e.g. table not found, in some scenario the callback will silently drop the error, not notifying the RPC which triggered this GetTableLocations call, causing the RPC to hang. This usually happens when user is inserting data, but underlying table is deleted and recreated. Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 --- M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java A java/kudu-client/src/main/java/org/kududb/client/ListTabletsRequest.java A java/kudu-client/src/main/java/org/kududb/client/ListTabletsResponse.java M java/kudu-client/src/test/java/org/kududb/client/TestAsyncKuduSession.java 4 files changed, 159 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/3295/3 -- To view, visit http://gerrit.cloudera.org:8080/3295 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Binglin ChangGerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] [java client] Fix GetTableLocations error handling
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3295 to look at the new patch set (#2). Change subject: [java client] Fix GetTableLocations error handling .. [java client] Fix GetTableLocations error handling When GetTableLocations get an error, e.g. table not found, in some scenario the callback will silently drop the error, not notifying the RPC which triggered this GetTableLocations call, causing the RPC to hang. This usually happens when user is inserting data, but underlying table is deleted and recreated. Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 --- M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java M java/kudu-client/src/test/java/org/kududb/client/TestAsyncKuduSession.java 2 files changed, 34 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/3295/2 -- To view, visit http://gerrit.cloudera.org:8080/3295 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Binglin ChangGerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins