ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Evdokimov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-379) Need to update CacheStore and CacheStoreSession APIs for conssitency
Date Tue, 03 Mar 2015 15:56:04 GMT

    [ https://issues.apache.org/jira/browse/IGNITE-379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14345241#comment-14345241

Sergey Evdokimov commented on IGNITE-379:

Also need to cleanup public API on Transaction: remove methods meta*()

> 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 <dsetrakyan@apache.org> 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

View raw message