This is an automated email from the ASF dual-hosted git repository.
andor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zookeeper.git
The following commit(s) were added to refs/heads/master by this push:
new 9a7cebb ZOOKEEPER-3314: Document the possibility of MultiCallback
receiving a null pointer
9a7cebb is described below
commit 9a7cebb96a8fe810fbeea18d911212076424feaa
Author: Steven McDonald <[email protected]>
AuthorDate: Tue Apr 9 14:36:10 2019 +0200
ZOOKEEPER-3314: Document the possibility of MultiCallback receiving a null
pointer
I have tried to get set up to verify that tests pass, but I am still seeing
numerous test failures when running `ant test` against the unmodified `master`
branch, so I cannot validate that this does not cause failures. Since this
particular change is only in documentation, it seems unlikely, so I'm raising a
PR anyway.
Author: Steven McDonald <[email protected]>
Reviewers: [email protected]
Closes #884 from stevenjm/ZOOKEEPER-3314
---
.../src/main/java/org/apache/zookeeper/AsyncCallback.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java
b/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java
index a0a0e1f..10f85da 100644
--- a/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java
+++ b/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java
@@ -326,9 +326,9 @@ public interface AsyncCallback {
* <p/>
* On failure, rc is a failure code in
* {@link org.apache.zookeeper.KeeperException.Code}.
- * All opResults are
+ * Either opResults is null, or all opResults are
* {@link org.apache.zookeeper.OpResult.ErrorResult}.
- * All operations will be rollback-ed even if operations
+ * All operations will be rolled back even if operations
* before the failing one were successful.
*
* @param rc The return code or the result of the call.
