[IGNITE-13170] Java thin client: Transactions functionality withLabel is broken

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.9
• Component/s: thin client
• Ignite Flags: Release Notes Required
• Assignee: Aleksey Plekhanov
• Reporter: Aleksey Plekhanov
• Created: 22/Jun/20 03:22
• Updated: 22/Jun/20 15:23
• Resolved: 22/Jun/20 15:23

Description

Reproducer

@Test
public void testTransactions1() throws Exception {
    try (Ignite ignite = Ignition.start(Config.getServerConfiguration());
         IgniteClient client = Ignition.startClient(getClientConfiguration()))
    {
        ClientCache<Integer, String> cache = client.createCache(new ClientCacheConfiguration()
                .setName("cache")
                .setAtomicityMode(CacheAtomicityMode.TRANSACTIONAL));

        cache.put(1, "value1");

        try (ClientTransaction tx = client.transactions().withLabel("asdasda").txStart()) {
            cache.put(1, "value2");
        }

        assertEquals("value1", cache.get(1));
    }
}

Root cause: a new instance of transactions facade is created when withLabel modificator is used. Transactions are registered in threadLocTxUid map of this instance, but when any cache operation is performed transaction is looked only at root threadLocTxUid map.

Issue Links

• links to: GitHub Pull Request #7951