[
https://issues.apache.org/jira/browse/IGNITE-1102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15233534#comment-15233534
]
Saikat Maitra commented on IGNITE-1102:
---------------------------------------
[~vkulichenko]
Hi
I have raised a PR for this change.
https://github.com/apache/ignite/pull/628
Also can you please share more info on the replacing assertion with
IllegalStateException change. Do we need to remove the
assert with If block and then throw IllegalStateException if condition not met.
Regards
Saikat
> IgniteSpringBean should have getConfiguration() method
> ------------------------------------------------------
>
> Key: IGNITE-1102
> URL: https://issues.apache.org/jira/browse/IGNITE-1102
> Project: Ignite
> Issue Type: Bug
> Components: general
> Affects Versions: 1.1.4
> Reporter: Valentin Kulichenko
> Assignee: Saikat Maitra
> Priority: Minor
> Labels: newbie
>
> This is required for proper integration with Spring. See forum thread:
> http://apache-ignite-users.70518.x6.nabble.com/IgniteSpringBean-getter-method-for-IgniteConfiguration-td613.html
> I would also replace assertions with IllegalStateException throwing (same
> conditions and messages) since assertions looks weird on top level API class.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
