This is an automated email from the ASF dual-hosted git repository.

ivank 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 0143075  [MERGE YAHOO REPO] Fixed NPE when accessing readonly bookie 
list
0143075 is described below

commit 01430750ce5a2f7c46cce942a7aa6116e41b9508
Author: Matteo Merli <[email protected]>
AuthorDate: Fri Jan 26 18:16:35 2018 +0100

    [MERGE YAHOO REPO] Fixed NPE when accessing readonly bookie list
    
    Descriptions of the changes in this PR:
    This is cherry-pick from yahoo repo of branch yahoo-4.3.
    
    original commit is:
    https://github.com/yahoo/bookkeeper/commit/270b69e4
    Fixed NPE when accessing readonly bookie list
    
    Author: Matteo Merli <[email protected]>
    
    Reviewers: Ivan Kelly <[email protected]>, Sijie Guo <[email protected]>
    
    This closes #1058 from jiazhai/cherry_picks/i_165
---
 .../apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java
index 24460d5..7dcd728 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java
@@ -192,7 +192,8 @@ class RackawareEnsemblePlacementPolicyImpl extends 
TopologyAwareEnsemblePlacemen
     protected Cache<BookieSocketAddress, Long> slowBookies;
     protected BookieNode localNode;
     protected final ReentrantReadWriteLock rwLock;
-    protected ImmutableSet<BookieSocketAddress> readOnlyBookies = null;
+    // Initialize to empty set
+    protected ImmutableSet<BookieSocketAddress> readOnlyBookies = 
ImmutableSet.of();
     protected boolean reorderReadsRandom = false;
     protected boolean enforceDurability = false;
     protected int stabilizePeriodSeconds = 0;

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to