[core] Fix the KvQueryTableTest may stuck in loop

[Aitozi]

Purpose

Before, it use the pattern as bellow, which will lead the ConcurrentHashMap stuck in loop

Map<String, String> test = new ConcurrentHashMap<>();
test.computeIfAbsent(
        "a",
        new Function<String, String>() {
            @Override
            public String apply(String s) {
                return test.remove("a");
            }
        });
System.out.println(test);

The KvQueryTableTest will stuck in the thread as below

Tests

API and Format

Documentation

[wwj6591812]

1、Does this must appear every time?
2、This is a bug in JDK?

[Aitozi]

1、Does this must appear every time? 2、This is a bug in JDK?

Map<String, String> test = new ConcurrentHashMap<>();
test.computeIfAbsent(
        "a",
        new Function<String, String>() {
            @Override
            public String apply(String s) {
                return test.remove("a");
            }
        });
System.out.println(test);

In jdk8 it will stuck, in jdk11 it will throw IllegalStageException. I think it's a wrong use case. So, we should avoid to modify the map in the computeIfAbsent call.

[JingsongLi]

+1