This is an automated email from the ASF dual-hosted git repository.
sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new b959f8f remove misleading comment
b959f8f is described below
commit b959f8f4d9363a11d27a28150b1fad54123d83cc
Author: Sijie Guo <[email protected]>
AuthorDate: Tue Dec 4 10:53:41 2018 -0800
remove misleading comment
Descriptions of the changes in this PR:
*Motivation*
ParallelRead is actually used internally at ledger recovery. It is used at
production.
*Changes*
Removed misleading comment.
Reviewers: Enrico Olivelli <[email protected]>, Jia Zhai <None>
This closes #1671 from sijie/remove_misleading_comment
---
.../src/main/java/org/apache/bookkeeper/client/PendingReadOp.java | 1 -
1 file changed, 1 deletion(-)
diff --git
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java
index aa89eaa..e820660 100644
---
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java
+++
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java
@@ -487,7 +487,6 @@ class PendingReadOp implements ReadEntryCallback,
SafeRunnable {
return speculativeTask;
}
- // I don't think this is ever used in production code -Ivan
PendingReadOp parallelRead(boolean enabled) {
this.parallelRead = enabled;
return this;
