GitHub user pavel-kuznetsov opened a pull request:
https://github.com/apache/ignite/pull/5657
IGNITE-10645: QueryBinaryProperty's isKey logic
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-10645
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/ignite/pull/5657.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 #5657
----
commit 42bb29bf24ab221c4a16655cce4c098bc65535b1
Author: Pavel Kuznetsov <palmihal@...>
Date: 2018-12-12T20:27:51Z
ignite-10645: Removed runtime property configuration.
Previously, in some cases (key is not of sql type and keyFields is null)
QueryBinaryProperty was configured at runtime (at first cache.put()).
Removed this logic, so now properties are configured when all binary
metadata gets processed.
commit 4aaaea81caa1d81ccd73b8e7b1d43ca34377b791
Author: Pavel Kuznetsov <palmihal@...>
Date: 2018-12-12T20:30:55Z
ignite-10645: Updated incorrect tests.
These tests used non-sql KeyObject, and expected that keyFields get
filled somehow (what is not supported). Added explicit keyFilds
declaration.
----
---
