You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Alberto Gomez (Jira)" <ji...@apache.org> on 2019/10/24 12:45:00 UTC

[jira] [Commented] (GEODE-6971) Update CqAttributesFactory to be a fluid API

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

Alberto Gomez commented on GEODE-6971:
--------------------------------------

Unfortunately, changing the return type from void to CqAttributesFactory in the addCqListener and initCqLIsteners methods breaks the backward compatibility of Geode as it can be observed when running the following upgrade test: org.apache.geode.security.ClientAuthorizationCQDUnitTest

The only way I can think of fixing this is creating new fluid versions of the old methods with a different name. The old ones could be deprecated in the future.

Is it worth it?

> Update CqAttributesFactory to be a fluid API
> --------------------------------------------
>
>                 Key: GEODE-6971
>                 URL: https://issues.apache.org/jira/browse/GEODE-6971
>             Project: Geode
>          Issue Type: Improvement
>          Components: core
>            Reporter: Charlie Black
>            Assignee: Alberto Gomez
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> CQAttributes does not currently support fluid API (like ClientCacheFactory and ClientRegionFactory). In the ideal world, I'd like to do something like the following
> {{CqAttributes cqa = new CqAttributesFactory()    }}{{   }}
> {{   .addCqListener(new SimpleCQListener())}}
> {{   .create();}}



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