timoninmaxim commented on code in PR #12569:
URL: https://github.com/apache/ignite/pull/12569#discussion_r2690317138
##########
modules/core/src/test/java/org/apache/ignite/internal/client/thin/TimeoutTest.java:
##########
@@ -217,4 +218,107 @@ public void testClientTimeoutOnOperation() throws
Exception {
}
}
}
+
+ /**
+ * Test that connection timeout is independent of request timeout during
connection establishment.
+ */
+ @Test
+ @SuppressWarnings("ThrowableNotThrown")
+ public void testConnectionTimeoutIndependentOfRequest() throws Exception {
+ ServerSocket sock = new ServerSocket();
+ sock.bind(new InetSocketAddress("127.0.0.1", DFLT_PORT));
+
+ CountDownLatch connectionAccepted = new CountDownLatch(1);
+
+ IgniteInternalFuture<?> fut = GridTestUtils.runAsync(() -> {
+ try {
+ Socket accepted = sock.accept();
+ connectionAccepted.countDown();
+
+ Thread.sleep(2000);
+
+ U.closeQuiet(accepted);
+ }
+ catch (Exception e) {
+ throw new IgniteException("Accept thread failed: " +
e.getMessage(), e);
+ }
+ });
+
+ try {
+ ClientConfiguration cfg = new ClientConfiguration()
+ .setAddresses("127.0.0.1:" + DFLT_PORT)
+ .setConnectionTimeout(500)
+ .setRequestTimeout(Integer.MAX_VALUE);
+
+ GridTestUtils.assertThrowsWithCause(
+ () -> Ignition.startClient(cfg),
+ IgniteFutureTimeoutCheckedException.class
+ );
+ }
+ finally {
+ U.closeQuiet(sock);
+ }
+
+ assertTrue("Connection should have been accepted",
connectionAccepted.await(1, TimeUnit.SECONDS));
+
+ fut.get();
+ }
+
+ /**
+ * Test that request timeout is independent of connection timeout during
operations.
+ */
+ @Test
+ @SuppressWarnings("ThrowableNotThrown")
+ public void testRequestTimeoutIndependentOfConnection() throws Exception {
+ IgniteConfiguration igniteCfg =
getConfiguration(getTestIgniteInstanceName());
+ igniteCfg.setClientConnectorConfiguration(new
ClientConnectorConfiguration().setHandshakeTimeout(Integer.MAX_VALUE));
+
+ try (Ignite ignite = startGrid(igniteCfg)) {
+ ClientConfiguration cfg = getClientConfiguration(ignite)
+ .setConnectionTimeout(Integer.MAX_VALUE)
+ .setRequestTimeout(500);
+
+ try (IgniteClient client = Ignition.startClient(cfg)) {
+ ClientCache<Object, Object> cache =
client.getOrCreateCache("testTimeoutCache");
+
+ ClientCacheConfiguration txCacheCfg = new
ClientCacheConfiguration()
+ .setName("txCache")
+ .setAtomicityMode(CacheAtomicityMode.TRANSACTIONAL);
+
+ ClientCache<Object, Object> txCache =
client.getOrCreateCache(txCacheCfg);
+
+ CyclicBarrier barrier = new CyclicBarrier(2);
+
+ IgniteInternalFuture<?> blockingThread =
GridTestUtils.runAsync(() -> {
+ try (ClientTransaction ignored1 =
client.transactions().txStart(PESSIMISTIC, REPEATABLE_READ)) {
+ txCache.put(1, "blocked");
+
+ barrier.await(2, TimeUnit.SECONDS);
+
+ // Wait for main thread to time out
+ barrier.await(2, TimeUnit.SECONDS);
+ }
+ catch (Exception e) {
+ throw new IgniteException(e);
+ }
+ });
+
+ barrier.await(2, TimeUnit.SECONDS);
+
+ try (ClientTransaction ignored1 =
client.transactions().txStart(PESSIMISTIC, REPEATABLE_READ)) {
+ GridTestUtils.assertThrowsWithCause(
+ () -> txCache.put(1, "should timeout"),
+ IgniteFutureTimeoutCheckedException.class
+ );
+ }
+
+ barrier.await(2, TimeUnit.SECONDS);
+
+ cache.put(2, "still works");
Review Comment:
2 -> 1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
