>From Ian Maxon <[email protected]>: Ian Maxon has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21155?usp=email )
Change subject: [NO ISSUE] Disable NCServiceExecutionIT ...................................................................... [NO ISSUE] Disable NCServiceExecutionIT Details: There is an issue with transaction replay where it can result in excessive stack size when memory is highly constrained. The root cause of the memory becoming so tight to begin with needs to be looked into Change-Id: Ifa04a02b8d950ae6e283c940e679c12e2dac3e48 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21155 Reviewed-by: Ali Alsuliman <[email protected]> Reviewed-by: Ian Maxon <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> --- M asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/NCServiceExecutionIT.java M asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Ali Alsuliman: Looks good to me, approved Jenkins: Verified; Verified Ian Maxon: Looks good to me, but someone else must approve Objections: Anon. E. Moose #1000171: Violations found diff --git a/asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/NCServiceExecutionIT.java b/asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/NCServiceExecutionIT.java index 58ecabd..3e085f8 100644 --- a/asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/NCServiceExecutionIT.java +++ b/asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/NCServiceExecutionIT.java @@ -47,6 +47,7 @@ import org.junit.Before; import org.junit.BeforeClass; import org.junit.FixMethodOrder; +import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.MethodSorters; @@ -201,7 +202,7 @@ testArgs.add(new Object[] { ctx, ctx, null }); } // let's kill something every 50 tests - if (testArgs.size() % 150 == 0) { + if (testArgs.size() % 50 == 0) { final KillCommand killCommand = KillCommand.values()[random.nextInt(KillCommand.values().length)]; testArgs.add(new Object[] { killCommand, null, killCommand }); } @@ -252,6 +253,8 @@ } @Test + @Ignore + // ignore for now because something is causing the NC to get into a memory starvation loop on recovery intermittently public void test() throws Exception { if (tcCtx != null) { testExecutor.executeTest(ACTUAL_RESULTS_DIR.getPath(), tcCtx, null, false); diff --git a/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf b/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf index 5c28e90..12fb27a 100644 --- a/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf +++ b/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf @@ -21,7 +21,7 @@ iodevices=../asterix-server/target/tmp/asterix_nc1/iodevice1,../asterix-server/target/tmp/asterix_nc1/iodevice2 nc.api.port=19004 #jvm.args=-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5006 -jvm.args=-Xmx4g -Dfile.encoding=iso-8859-1 +jvm.args=-Xmx5g -Dfile.encoding=iso-8859-1 [nc/asterix_nc2] ncservice.port=9091 @@ -30,7 +30,7 @@ iodevices=../asterix-server/target/tmp/asterix_nc2/iodevice1,../asterix-server/target/tmp/asterix_nc2/iodevice2 nc.api.port=19005 #jvm.args=-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5007 -jvm.args=-Xmx4g -Dfile.encoding=us-ascii +jvm.args=-Xmx5g -Dfile.encoding=us-ascii [nc] address=127.0.0.1 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21155?usp=email To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: asterixdb Gerrit-Branch: lumina Gerrit-Change-Id: Ifa04a02b8d950ae6e283c940e679c12e2dac3e48 Gerrit-Change-Number: 21155 Gerrit-PatchSet: 7 Gerrit-Owner: Ian Maxon <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]>
