[jira] [Commented] (OFBIZ-9829) Add a new plugin for the redis

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-9829) Add a new plugin for the redis

JIRA jira@apache.org

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

Michael Brohl commented on OFBIZ-9829:
--------------------------------------

[~shi.jinghai],

my concern is not about Redis and it does not matter if I personally like it or not. In fact, I have no strong opinion about it at the moment, because I had not the chance to think about it.

My concern is about the way you commit a new plugin to the codebase without asking the community.

Apache OFBiz is a community project. Architectural changes, refactorings, changes of business logic and also the introduction of new functionality and enhancements should be explained, discussed, reviewed and approved with a consensus.

I kindly ask you to revert the commit and start a proposal to introduce a Redis integration for OFBiz.

If you have good arguments for adding it and if the solution does fit in the sense of quality, user benefit and documentation, I'm pretty sure it will find it's way to the codebase.

Thanks,
Michael


> Add a new plugin for the redis
> ------------------------------
>
>                 Key: OFBIZ-9829
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9829
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: Trunk
>            Reporter: Hadwin Dai
>            Assignee: Shi Jinghai
>              Labels: redis
>         Attachments: OFBIZ-9829_redis-plugins.patch
>
>
> Redis is an open source (BSD licensed), in-memory data structure store, used as a database, cache and message broker.
> Included services: get data, set data and delete data.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)