[ https://issues.apache.org/jira/browse/IGNITE-5211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116541#comment-16116541 ]
ASF GitHub Bot commented on IGNITE-5211: ---------------------------------------- GitHub user tledkov-gridgain opened a pull request: https://github.com/apache/ignite/pull/2407 IGNITE-5211 Classes based constructor for QueryEntities You can merge this pull request into a Git repository by running: $ git pull https://github.com/gridgain/apache-ignite ignite-5211 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/ignite/pull/2407.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2407 ---- commit f027c03167183254768c24c0aca4a8ddc33b985a Author: tledkov-gridgain <tled...@gridgain.com> Date: 2017-08-03T14:36:23Z IGNITE-5211 Classes based constructor for QueryEntities commit e8a7317787a6b89e793745b37c53a8b6b9939136 Author: tledkov-gridgain <tled...@gridgain.com> Date: 2017-08-03T14:48:01Z IGNITE-5211 minors commit 4aa27a1efc75703a2fc40ba54e8a23ad2f4ff0cc Author: tledkov-gridgain <tled...@gridgain.com> Date: 2017-08-07T09:49:03Z Merge branch '_master' into ignite-5211 commit 91090b597a3d6a026b20b3477142376046c1808b Author: tledkov-gridgain <tled...@gridgain.com> Date: 2017-08-07T09:52:36Z Merge branch '_master' into ignite-5211 commit e8e850607a85152429a23fe72da2a0d5bbbe018c Author: devozerov <voze...@gridgain.com> Date: 2017-08-07T10:53:56Z Cosmetics. commit 89c47cc290821db18b7731dace70d095829cdfcb Author: tledkov-gridgain <tled...@gridgain.com> Date: 2017-08-07T12:23:36Z IGNITE-5211: fix review comments commit ca191457fe554c39a1395d778bfe119477f3b5a8 Author: devozerov <voze...@gridgain.com> Date: 2017-08-07T12:45:09Z Removing unused classes. ---- > Classes based constructor for QueryEntities > ------------------------------------------- > > Key: IGNITE-5211 > URL: https://issues.apache.org/jira/browse/IGNITE-5211 > Project: Ignite > Issue Type: Task > Components: sql > Reporter: Denis Magda > Assignee: Taras Ledkov > Priority: Critical > Fix For: 2.2 > > > We need to add constructor {{QueryEntity(Class keyType, Class valueType)}} to > query entities class and deprecate {{CacheConfiguration.setIndexedTypes(…)}} > method. > Otherwise, there is no easy way for people who define SQL scheme with the > annotations to do advanced settings like names of key and value objects, > table name, etc. > See this discussion for more details: > http://apache-ignite-developers.2346864.n4.nabble.com/SQL-setting-key-field-name-for-types-registered-via-CacheConfiguration-setIndexedTypes-td17603.html -- This message was sent by Atlassian JIRA (v6.4.14#64029)