[
https://issues.apache.org/jira/browse/IGNITE-379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14348330#comment-14348330
]
Semen Boikov commented on IGNITE-379:
-------------------------------------
After discussion we decided to leave CacheStoreSession.properties() as is, just
need to add test to make sure session is injected into all child classes if
they have field with CacheStoreSessionResource annotation.
> 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)
