[GitHub] ignite pull request #5716: IGNITE-9989

[pavel-kuznetsov <gi...@git.apache.org>]

GitHub user pavel-kuznetsov opened a pull request:

    https://github.com/apache/ignite/pull/5716

    IGNITE-9989

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gridgain/apache-ignite ignite-9989-new2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/ignite/pull/5716.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 #5716
    
----
commit 42bb29bf24ab221c4a16655cce4c098bc65535b1
Author: Pavel Kuznetsov <pa...@...>
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 <pa...@...>
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.

commit acfb97f0345a622bf4e2b948cdb7236ce6169b8c
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-13T10:17:45Z

    ignite-10645: Removed garbage; Fixed test.
    
    Removed accidentally added test class. Fixed configuration of forgotten
    test.

commit 21045b5f0bd4f5701b92518bd1289a54107354bb
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-13T14:53:26Z

    ignite-10645: Updated doc.

commit bed7b76ade5990f4392e1169b060ef82dc67f3fe
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-14T16:53:26Z

    ignite-10645: Updated comment. Simplified condition.
    
    Since we don't decide is property a "key-property" at runtime, condition
     for isKeyProp could be simplified.
    Also corrected comment about binary property creation.

commit e438c5ab535924f967b0a203c3646720e9596340
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-17T02:11:29Z

    ignite-10645: Added validation of insert queries.
    
    Insert query should contain all primary key columns.

commit 824e7531a58abbed31250f870467f645bf324977
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-17T02:18:24Z

    ignite-10645: Added tests for insert query validation.

commit 681e4452e31f6f6e318e2edf7c6b4831baa6eb55
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-17T19:37:37Z

    ignite-10645: Simplified validation. Updated tests.
    
    Decided validate inserts only for cases "no key columns specified" and
    "_key mixed with other key columns".

commit 85f36b18342e82b25945c1294484082692c93f73
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-18T12:50:03Z

    Merge remote-tracking branch 'origin/master' into ignite-10645

commit e4f340b55d7a8e7da0ebc2d5d7e14ba92a6f5ba2
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-18T12:52:25Z

    ignite-10645: fix import after merge.

commit 556d584da6a235643fb48e9fb69c0aa2794dd1df
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-18T14:13:09Z

    ignite-10645: Fixed test.
    
    This test fails only on teamcity due to locally it does have Indexing
    in the classpath. On TC it does. So in local run QueryEntity it is not
    validated.
    Since validation is not scope of these tests, just fixed QueryEntity
    configuration which was not correct.

commit 1852e027aea67c6b822370395ddec03ab5c9e53c
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-11-06T14:39:11Z

    ignite-9989: Updated tests to work with many tables.
    
    Backported test for primary keys metadata from metadata test of jdbc
    thin driver. This change made IGNITE-10118 highlighted.

commit 6bfa27b57b978247bfa4998e7523bb361107ed56
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-07T15:34:57Z

    ignite-9989: jdbc thin metadata logic for jdbcv2 driver.
    
    Imported logic of getting primary keys from thin driver to v2 driver.

commit 28a6c34748770aa97836d0452d89581f4b37b67d
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-18T18:30:01Z

    ignite-9989: fixed test.
    
    New table in the test setup have been added, updated the test.

commit a5c85628c7829b71bcf5758766ecd48ee2f5c92d
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-18T18:47:59Z

    ignite-9989: Unified jdbcv2 and thin keys metadata logic.

commit 1022379a9cf35f02b170b003251fd023c5b8464b
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-19T16:06:11Z

    ignite-9989: Unified jdbcv2 and thin getTables metadata logic.

commit 69ef2deb26043823ab6e84db78c80746cf6a7f5a
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-19T16:07:30Z

    ignite-9989: Ensure not closed on getPKs in jdbc v2.

commit d7bf5b4981598cba438fc3f8c92fedb30bb8980b
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-19T22:57:43Z

    ignite-9989: Unified getColumns metadata v2 and thin driver logic.

commit 2efe713fd516c41311d970edc75648d9894566fb
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-19T23:31:59Z

    ignite-9989: Unified getSchemas metadata v2 and thin driver logic.
    
    This commit also fixes IGNITE-10118.

commit dd47685165c825607fa7a6d2fe3601cd0530457f
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-20T12:20:20Z

    ignite-9989: Set -> SortedSet signature for getSchemas
    
    getSchemas has contract, that results are sorted by (catalog, schema).
    Changed helper method signature to eliminate that.

commit 80ba63add0c9c43feb369eb90f36567e955d3102
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-20T15:23:13Z

    ignite-9989: unified v2 and thin metadata logic for getIndexInfo

commit 54ca30fc705643472eda33e09987f1aa72e903c2
Author: Pavel Kuznetsov <pa...@...>
Date:   2018-12-20T23:14:31Z

    ignite-9989: Completely removed old metadata logic in the v2 driver.

----


---