[
https://issues.apache.org/jira/browse/IGNITE-379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sergey Evdokimov resolved IGNITE-379.
-------------------------------------
Resolution: Fixed
> Need to update CacheStore and CacheStoreSession APIs for conssitency
> --------------------------------------------------------------------
>
> Key: IGNITE-379
> URL: https://issues.apache.org/jira/browse/IGNITE-379
> Project: Ignite
> Issue Type: Bug
> Components: cache
> Reporter: Dmitriy Setrakyan
> Assignee: Sergey Evdokimov
> Priority: Blocker
> Fix For: sprint-2
>
>
> From mailing archives:
> {noformat}
> In the "Persistent Store" documentation I have tried the API changes I
> suggested, and I think it looks good and consistent:
> http://apacheignite.readme.io/v1.0/docs/persistent-store
> Would be nice if we could make these changes in sprint-2. They are very
> minimal.
> I have also asked Andrey N and Alexey K to document the automatic DB schema
> integration, which is a big differentiating feature for Ignite.
> Hopefully we will have complete Data Grid documentation next week.
> D.
> On Sat, Feb 28, 2015 at 10:03 AM, Dmitriy Setrakyan <[email protected]>
> wrote:
> I am looking at the store and how it interacts with the session and think we
> should make some changes:
> 1. There is no need to have a HashMap for session properties. We can just
> have attach(Object) and getAttached(). If user wants to attach a HashMap
> there, he still can.
> 2. I don't think CacheStore.txEnd() method is correct. We should call it
> CacheStore.sessionEnd(). For non-transactional operations, this method should
> be invoked after each call, and for transactional operations, it should be
> invoked only after transaction completed.
> 3. I also have added isWithinTransaction() method to the session for
> convenience.
> I think the above change will allow us to have the same session semantics
> between transactional and non-transactional caches.
> Thoughts?
> D.
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
