This is an automated email from the ASF dual-hosted git repository. jchen21 pushed a commit to branch feature/GEODE-5301 in repository https://gitbox.apache.org/repos/asf/geode.git
commit fe9f0d9176c8245462f605ddfbe0475ddc2f8a5a Author: Jianxia Chen <[email protected]> AuthorDate: Thu Jun 7 16:27:52 2018 -0700 Check readiness and closed before assertion [GEODE-5301] --- .../java/org/apache/geode/internal/cache/ProxyBucketRegion.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/ProxyBucketRegion.java b/geode-core/src/main/java/org/apache/geode/internal/cache/ProxyBucketRegion.java index 3a256a9..21ff006 100755 --- a/geode-core/src/main/java/org/apache/geode/internal/cache/ProxyBucketRegion.java +++ b/geode-core/src/main/java/org/apache/geode/internal/cache/ProxyBucketRegion.java @@ -230,15 +230,15 @@ public class ProxyBucketRegion implements Bucket { * @param br the real bucket which will be the target for this proxy */ public void setBucketRegion(BucketRegion br) { - Assert.assertTrue(this.realBucket == null); - Assert.assertTrue(!this.advisor.isHosting()); - this.realBucket = br; // fix several bugs including 36881... creation of BR may be occurring // at same time another thread is destroying the PR and now that this // BR is visible to the destroy thread we want to prevent sending bogus // CreateRegion or profile update messages this.partitionedRegion.checkReadiness(); this.partitionedRegion.checkClosed(); + Assert.assertTrue(this.realBucket == null); + Assert.assertTrue(!this.advisor.isHosting()); + this.realBucket = br; } public void clearBucketRegion(BucketRegion br) { -- To stop receiving notification emails like this one, please contact [email protected].
