[jira] [Commented] (GEODE-7824) Revert Public API change to ClientRegionFactory

[Lynn Hughes-Godfrey (Jira)]

    [ 
https://issues.apache.org/jira/browse/GEODE-7824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17047030#comment-17047030
 ] 

Lynn Hughes-Godfrey commented on GEODE-7824:
--------------------------------------------

commit a2ac820116c7a54dcbf95dbaa0fd37eb5e7e09d8
Author: Kirk Lund <kl...@apache.org>
Date:   Fri Feb 7 11:58:57 2020 -0800

    GEODE-5595: Fix DeltaPropagationDUnitTest flakiness (#4653)

    Improve testability of CacheClientProxy
    * Extract inner classes
    * Introduce CacheClientProxyFactory with support for property injection


diff --git 
a/geode-core/src/main/java/org/apache/geode/cache/client/internal/ClientRegionFactoryImpl.java
 
b/geode-core/src/main/java/org/apache/geode/cache/client/internal/ClientRegionFactoryImpl.java
index 64256e8f8e..920deba055 100644
--- 
a/geode-core/src/main/java/org/apache/geode/cache/client/internal/ClientRegionFactoryImpl.java
+++ 
b/geode-core/src/main/java/org/apache/geode/cache/client/internal/ClientRegionFactoryImpl.java
@@ -186,8 +186,9 @@ public class ClientRegionFactoryImpl<K, V> implements 
ClientRegionFactory<K, V>
   }
 
   @Override
-  public void setConcurrencyChecksEnabled(boolean concurrencyChecksEnabled) {
+  public ClientRegionFactory<K, V> setConcurrencyChecksEnabled(boolean 
concurrencyChecksEnabled) {
     this.attrsFactory.setConcurrencyChecksEnabled(concurrencyChecksEnabled);
+    return this;
   }


> Revert Public API change to ClientRegionFactory
> -----------------------------------------------
>
>                 Key: GEODE-7824
>                 URL: https://issues.apache.org/jira/browse/GEODE-7824
>             Project: Geode
>          Issue Type: Bug
>          Components: regions
>    Affects Versions: 1.13.0
>            Reporter: Udo Kohlmeyer
>            Assignee: Kirk Lund
>            Priority: Major
>
> As per a commit to the ClientRegionFactory, the public API was changed.
> Even though it was to get some consistency in the class, it does now deviate 
> from the released public API.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)