Murtadha Hubail has submitted this change and it was merged.

Change subject: [ASTERIXDB-232]6[TEST] Increase Result TTL to 5 Mins
......................................................................


[ASTERIXDB-232]6[TEST] Increase Result TTL to 5 Mins

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Increase result TTL to 5 mins in AsterixHyracksIntegrationUtil
  to avoid intermittent test failures due to long GC pauses in
  long running queries.

Change-Id: I6ecf0e4ac3578027ed12ac9d34d8b2b8f04b348b
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2526
Reviewed-by: Michael Blow <mb...@apache.org>
Integration-Tests: Michael Blow <mb...@apache.org>
Tested-by: Michael Blow <mb...@apache.org>
---
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Anon. E. Moose #1000171: 
  Michael Blow: Looks good to me, approved; Verified; Verified



diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
index acc3970..cc70d57 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
@@ -30,6 +30,7 @@
 import java.nio.file.attribute.BasicFileAttributes;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.concurrent.TimeUnit;
 import java.util.function.BiPredicate;
 import java.util.stream.Stream;
 
@@ -70,6 +71,7 @@
             joinPath(getProjectPath().toString(), "src", "test", "resources", 
"cc.conf");
     private static final String DEFAULT_STORAGE_PATH = joinPath("target", 
"io", "dir");
     private static String storagePath = DEFAULT_STORAGE_PATH;
+    private static final long RESULT_TTL = TimeUnit.MINUTES.toMillis(5);
 
     static {
         System.setProperty("java.util.logging.manager", 
org.apache.logging.log4j.jul.LogManager.class.getName());
@@ -198,7 +200,7 @@
         
ccConfig.setClientListenAddress(Inet4Address.getLoopbackAddress().getHostAddress());
         ccConfig.setClientListenPort(DEFAULT_HYRACKS_CC_CLIENT_PORT);
         ccConfig.setClusterListenPort(DEFAULT_HYRACKS_CC_CLUSTER_PORT);
-        ccConfig.setResultTTL(120000L);
+        ccConfig.setResultTTL(RESULT_TTL);
         ccConfig.setResultSweepThreshold(1000L);
         ccConfig.setEnforceFrameWriterProtocol(true);
         configManager.set(ControllerConfig.Option.DEFAULT_DIR, 
joinPath(getDefaultStoragePath(), "asterixdb"));
@@ -217,7 +219,7 @@
         
ncConfig.setDataListenAddress(Inet4Address.getLoopbackAddress().getHostAddress());
         
ncConfig.setResultListenAddress(Inet4Address.getLoopbackAddress().getHostAddress());
         
ncConfig.setMessagingListenAddress(Inet4Address.getLoopbackAddress().getHostAddress());
-        ncConfig.setResultTTL(120000L);
+        ncConfig.setResultTTL(RESULT_TTL);
         ncConfig.setResultSweepThreshold(1000L);
         ncConfig.setVirtualNC();
         configManager.set(ControllerConfig.Option.DEFAULT_DIR, 
joinPath(getDefaultStoragePath(), "asterixdb", ncName));

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2526
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6ecf0e4ac3578027ed12ac9d34d8b2b8f04b348b
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <mhub...@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Murtadha Hubail <mhub...@apache.org>

Reply via email to