[jira] [Comment Edited] (CAY-1873) Move DataDomain cache configuration from the Modeler and into DI

["Oleg Artyukevich (JIRA)" <ji...@apache.org>]

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

Oleg Artyukevich edited comment on CAY-1873 at 2/18/14 8:15 AM:
----------------------------------------------------------------

Hi, Andrus. 
I added annotations and licence headers in tests and server part of patch. If I understand correctly, "modeler patch" should not have any additional changes? I just delete cache configuration there.


was (Author: olegart):
Hi, Andrus. 
I added annotations and licence headers in tests and server part of patch. If I understand correctly, "modeler patch" should have any additional changes? I just delete cache configuration there.

> Move DataDomain cache configuration from the Modeler and into DI
> ----------------------------------------------------------------
>
>                 Key: CAY-1873
>                 URL: https://issues.apache.org/jira/browse/CAY-1873
>             Project: Cayenne
>          Issue Type: Task
>            Reporter: Andrus Adamchik
>            Assignee: Andrus Adamchik
>         Attachments: cay1873.patch, cay1873_modeler.patch, cay1873_test.patch, cay1873v2.patch, cay1873v2_test.patch, x.png
>
>
> Cache configuration for DataDomain is currently done in the Modeler. (this configuration is limited to the "object cache" [1], query cache is configured in DI already)  and this seems misplaced. Cache settings are highly environment dependent and are better managed by DI. So let's remove all these screens from the Modeler and create any needed DI services that perform this configuration function.
> (The next step, outside the scope of this Jira, may be combining query cache and object cache in the same cache engine)
> [1] http://cayenne.apache.org/docs/3.0/caching-and-fresh-data.html



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)