[IGNITE-5227] StackOverflowError in GridCacheMapEntry#checkOwnerChanged()

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.6
• Fix Version/s: 2.8
• Assignee: Stepachev Maksim
• Reporter: Alexey Goncharuk
• Created: 15/May/17 14:17
• Updated: 20/Aug/19 16:31
• Resolved: 01/Aug/19 16:53

Description

A simple test reproducing this error:

    /**
     * @throws Exception if failed.
     */
    public void testBatchUnlock() throws Exception {
       startGrid(0);
       grid(0).createCache(new CacheConfiguration<Integer, Integer>(DEFAULT_CACHE_NAME)
            .setAtomicityMode(CacheAtomicityMode.TRANSACTIONAL));

        try {
            final CountDownLatch releaseLatch = new CountDownLatch(1);

            IgniteInternalFuture<Object> fut = GridTestUtils.runAsync(new Callable<Object>() {
                @Override public Object call() throws Exception {
                    IgniteCache<Object, Object> cache = grid(0).cache(null);

                    Lock lock = cache.lock("key");

                    try {
                        lock.lock();

                        releaseLatch.await();
                    }
                    finally {
                        lock.unlock();
                    }

                    return null;
                }
            });

            Map<String, String> putMap = new LinkedHashMap<>();

            putMap.put("key", "trigger");

            for (int i = 0; i < 10_000; i++)
                putMap.put("key-" + i, "value");

            IgniteCache<Object, Object> asyncCache = grid(0).cache(null).withAsync();

            asyncCache.putAll(putMap);

            IgniteFuture<Object> resFut = asyncCache.future();

            Thread.sleep(1000);

            releaseLatch.countDown();

            fut.get();

            resFut.get();
        }
        finally {
            stopAllGrids();
        }

We should replace a recursive call with a simple iteration over the linked list.

Issue Links

• is related to: IGNITE-12087 Transactional putAll - significant performance drop on big batches of entries.
• links to: GitHub Pull Request #1967
• links to: GitHub Pull Request #1967
• links to: GitHub Pull Request #1972
• links to: GitHub Pull Request #6736