[IGNITE-6307] If getAll() fails with NPE, onHeap entry is not removed, for local cache

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0
• Fix Version/s: 2.4
• Component/s: cache
• Labels: MakeTeamcityGreenAgain
• Assignee: Nikolay Izhikov
• Reporter: Dmitriy Govorukhin
• Created: 08/Sep/17 08:09
• Updated: 05/Oct/17 03:42
• Resolved: 04/Oct/17 16:32

Description

GridCacheLocalFullApiSelfTest.testGetAllWithNulls

    final Set<String> c = new HashSet<>();

        c.add("key1");
        c.add(null);

        GridTestUtils.assertThrows(log, new Callable<Void>() {
            @Override public Void call() throws Exception {
                cache.getAll(c);

                return null;
            }
        }, NullPointerException.class, null);

After getAll, entry with "key1" will be in heap map, it is dependent of order in collection for getAll();

Issue Links

• links to: GitHub Pull Request #2629
• links to: GitHub Pull Request #2661