You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Mikhail Petrov <pm...@gmail.com> on 2021/10/04 15:51:22 UTC
Migration of CacheSpringStoreSessionListener to ignite-extensions
Igniters, I propose to migrate CacheSpringStoreSessionListener to
ignite-extensions in the ignite-spring-tx module as part of Spring
Transactions integration migration. It also helps to
1. Make the ignite-spring module responsible only for parsing XML
configurations.
2. Reduce the number of dependencies that the ignite-spring module
relies on to just spring-context.
PRs with the migration - [1] and [2].
Any objections?
[1] - https://github.com/apache/ignite-extensions/pull/74
[2] - https://github.com/apache/ignite/pull/9467
--
Mikhail
Re: Migration of CacheSpringStoreSessionListener to ignite-extensions
Posted by Ivan Daschinsky <iv...@gmail.com>.
+1 As for me. It is pretty weird that this few lines of code depend on full
spring :)
вт, 5 окт. 2021 г. в 11:24, Nikita Amelchev <na...@apache.org>:
> Mikhail,
>
> +1 to migrate and include to the ignite-spring-tx 1.0.0, Ignite 2.12 scopes
>
> Please, create an issue.
>
> пн, 4 окт. 2021 г. в 18:51, Mikhail Petrov <pm...@gmail.com>:
> >
> > Igniters, I propose to migrate CacheSpringStoreSessionListener to
> > ignite-extensions in the ignite-spring-tx module as part of Spring
> > Transactions integration migration. It also helps to
> > 1. Make the ignite-spring module responsible only for parsing XML
> > configurations.
> > 2. Reduce the number of dependencies that the ignite-spring module
> > relies on to just spring-context.
> >
> > PRs with the migration - [1] and [2].
> >
> > Any objections?
> >
> > [1] - https://github.com/apache/ignite-extensions/pull/74
> > [2] - https://github.com/apache/ignite/pull/9467
> >
> > --
> > Mikhail
> >
>
>
> --
> Best wishes,
> Amelchev Nikita
>
--
Sincerely yours, Ivan Daschinskiy
Re: Migration of CacheSpringStoreSessionListener to ignite-extensions
Posted by Nikita Amelchev <na...@apache.org>.
Mikhail,
+1 to migrate and include to the ignite-spring-tx 1.0.0, Ignite 2.12 scopes
Please, create an issue.
пн, 4 окт. 2021 г. в 18:51, Mikhail Petrov <pm...@gmail.com>:
>
> Igniters, I propose to migrate CacheSpringStoreSessionListener to
> ignite-extensions in the ignite-spring-tx module as part of Spring
> Transactions integration migration. It also helps to
> 1. Make the ignite-spring module responsible only for parsing XML
> configurations.
> 2. Reduce the number of dependencies that the ignite-spring module
> relies on to just spring-context.
>
> PRs with the migration - [1] and [2].
>
> Any objections?
>
> [1] - https://github.com/apache/ignite-extensions/pull/74
> [2] - https://github.com/apache/ignite/pull/9467
>
> --
> Mikhail
>
--
Best wishes,
Amelchev Nikita