Terrible performance for some filters

[x31xx]

Describe the bug
When upgrading from v14 to v24 we noticed that the performance for GreaterFilter, LessFilter and LikeFilters among others was more than 100+ times worse. I reproduced this in a local test. It seems like the ChainedCollection is the problem.

To Reproduce
Attached a test that demonstrates the problem:

`
import com.oracle.coherence.common.collections.NullableSortedMap;
import com.tangosol.util.Binary;
import com.tangosol.util.ChainedCollection;
import com.tangosol.util.SubSet;
import org.junit.Assert;
import org.junit.Test;

import java.nio.charset.StandardCharsets;
import java.util.*;

public class TestCoherence {

@Test
public void testCoherenceRetainAll()   {
    SubSet<Binary> setKeys = new SubSet<>(createSet());
    SubSet<Binary> setKeys2 = new SubSet<>(setKeys);
    NullableSortedMap<Integer, Set<Binary>> mapTail = new NullableSortedMap<>();
    for (int i = 0; i < 43000; i++) {
        mapTail.put(i, Collections.singleton(createBinary(i)));
    }
    //New Style:
    long start = System.currentTimeMillis();
    List<Set<?>> listGT = new ArrayList<>(mapTail.size());
    for (Object o : mapTail.values()) {
        Set set = (Set) o;
        listGT.add(ensureSafeSet(set));
    }
    setKeys.retainAll(new ChainedCollection<>(listGT.toArray(Set[]::new)));
    long newStyle = System.currentTimeMillis() - start;
    start = System.currentTimeMillis();
    NullableSortedMap<Integer, Set<Binary>> mapGE = new NullableSortedMap<>(mapTail);
    //Old style:
    Set setGT = new HashSet();
    for (Iterator iterGE = mapGE.values().iterator(); iterGE.hasNext(); ) {
        Set set = (Set) iterGE.next();
        setGT.addAll(ensureSafeSet(set));

    }
    setKeys2.retainAll(setGT);
    long oldStyle = System.currentTimeMillis() - start;
    System.out.println("Time: new style(v24):" + newStyle + " ms, old style(v14):" + oldStyle + " ms");
    Assert.assertEquals(setKeys, setKeys2);
}

private Set<Binary> createSet() {
    Set<Binary> s = new HashSet<>();
    for (int i = 0; i < 30000; i++) {
        s.add(createBinary(i + 15000));
    }
    return s;
}

protected static Set ensureSafeSet(Set set) {
    return set == null ? Collections.emptySet() : set;
}

private Binary createBinary(int i) {
    String s = "randomkey_no_test_gldfkjglkdfgjdflkgjdflkgjlkdfjglkdfjglkdfjglkdf" + i;
    return new Binary(s.getBytes(StandardCharsets.UTF_8));

}

}
`

Expected behaviour
At least the same performance as before

Screenshots
If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

• Coherence CE 24.
• Java version Coretto 21
• OS: windows
• OS Version w11

Additional context
Add any other context about the problem here.

[mgamanho]

Hi, we're picking this up and will get back to you ASAP. Thanks

[mgamanho]

Thanks for bringing this up and a great reproducer. We're working on a fix, currently in testing.

[mgamanho]

Thanks again, this is fixed here in CE/main.

Other branches getting the fix are 25.03 and 22.06 as 24.xx is not LTS.

[x31xx]

Hello again, performance is much better but still not up to the speed in v14. Before the fix our big server test v24 was more than 100 times slower then v14. in v25.03.1 it is around 10 times slower then v14. We have a workaround, basically we took the sources for GreaterFilter; LessFilter etc from v 14 and and tweaked them so they compile in v25.

[mgamanho]

Thanks for the feedback. Would you mind sharing the changes you made to one of the filters? That would help expedite.

[mgamanho]

@x31xx I've created issue 141 for this. Please attach details to it, ideally a reproducer or description of one since a number of factors can make this hard to reproduce. Thank you