Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12237 )
Change subject: [java] log warning if applying operation on a closed session ...................................................................... [java] log warning if applying operation on a closed session Closed sessions won't flush on another close() call, so it's somewhat dubious to apply() an operation to a closed session. This change adds a warning if apply() is used with a closed session. Originally this patch enforced the new behavior via precondition, but that's a breaking change for clients that relied on the old broken behavior. Change-Id: I0fe221fedbb91959985f5ee374f1b691be2426a9 Reviewed-on: http://gerrit.cloudera.org:8080/12237 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java 2 files changed, 30 insertions(+), 2 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12237 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0fe221fedbb91959985f5ee374f1b691be2426a9 Gerrit-Change-Number: 12237 Gerrit-PatchSet: 7 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
