[jira] [Commented] (GEODE-706) CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy
[ https://issues.apache.org/jira/browse/GEODE-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15602779#comment-15602779 ] ASF subversion and git services commented on GEODE-706: --- Commit 24a72040cba3afe96da8ef752aa1f07c10dd1fa1 in incubator-geode's branch refs/heads/feature/GEODE-1985 from [~hitesh.khamesra] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=24a7204 ] GEODE-706 Fixed race condition between expiry thread and user thread. ExpirationManager tracks the regionEntry as reference to expiryTask. It assumes, it is unique for that key. But consistency check mechanism keeps that regionEntry around and that enforces re-use of that for new update. That introduces the race condition between expiry thread and user thread, it endup not scheduling the entry for expiration. As a fix, now "consistency check mechanism" unschedules the expiry task to avoid this race condition. Unmarked flaky from testEntryIdleDestroy test > CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy > - > > Key: GEODE-706 > URL: https://issues.apache.org/jira/browse/GEODE-706 > Project: Geode > Issue Type: Bug > Components: regions >Affects Versions: 1.0.0-incubating >Reporter: Jens Deppe >Assignee: Hitesh Khamesra > Labels: CI, Flaky > > git rev e3d24d77150382f363f334b3e2d6622c6e02e8bc build #948 > {noformat} > Error Message > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > Stacktrace > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > at com.gemstone.gemfire.internal.Assert.throwError(Assert.java:91) > at com.gemstone.gemfire.internal.Assert.assertTrue(Assert.java:115) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2074) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2034) > at > com.gemstone.gemfire.cache30.RegionTestCase.testEntryIdleDestroy(RegionTestCase.java:3569) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at junit.framework.TestCase.runTest(TestCase.java:176) > at junit.framework.TestCase.runBare(TestCase.java:141) > at junit.framework.TestResult$1.protect(TestResult.java:122) > at junit.framework.TestResult.runProtected(TestResult.java:142) > at junit.framework.TestResult.run(TestResult.java:125) > at junit.framework.TestCase.run(TestCase.java:129) > at junit.framework.TestSuite.runTest(TestSuite.java:252) > at junit.framework.TestSuite.run(TestSuite.java:247) > at > org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:86) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:105) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:56) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:64) > at > org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:50) > at sun.reflect.GeneratedMethodAccessor73.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32) > at > org.gradle.messaging.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93) > at com.sun.proxy.$Proxy2.processTestClass(Unknown Source) > at > org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:106) > at sun.reflect.GeneratedMethodAccessor72.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at >
[jira] [Commented] (GEODE-706) CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy
[ https://issues.apache.org/jira/browse/GEODE-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15597129#comment-15597129 ] ASF subversion and git services commented on GEODE-706: --- Commit 24a72040cba3afe96da8ef752aa1f07c10dd1fa1 in incubator-geode's branch refs/heads/develop from [~hitesh.khamesra] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=24a7204 ] GEODE-706 Fixed race condition between expiry thread and user thread. ExpirationManager tracks the regionEntry as reference to expiryTask. It assumes, it is unique for that key. But consistency check mechanism keeps that regionEntry around and that enforces re-use of that for new update. That introduces the race condition between expiry thread and user thread, it endup not scheduling the entry for expiration. As a fix, now "consistency check mechanism" unschedules the expiry task to avoid this race condition. Unmarked flaky from testEntryIdleDestroy test > CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy > - > > Key: GEODE-706 > URL: https://issues.apache.org/jira/browse/GEODE-706 > Project: Geode > Issue Type: Bug > Components: regions >Affects Versions: 1.0.0-incubating >Reporter: Jens Deppe >Assignee: Hitesh Khamesra > Labels: CI, Flaky > > git rev e3d24d77150382f363f334b3e2d6622c6e02e8bc build #948 > {noformat} > Error Message > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > Stacktrace > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > at com.gemstone.gemfire.internal.Assert.throwError(Assert.java:91) > at com.gemstone.gemfire.internal.Assert.assertTrue(Assert.java:115) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2074) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2034) > at > com.gemstone.gemfire.cache30.RegionTestCase.testEntryIdleDestroy(RegionTestCase.java:3569) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at junit.framework.TestCase.runTest(TestCase.java:176) > at junit.framework.TestCase.runBare(TestCase.java:141) > at junit.framework.TestResult$1.protect(TestResult.java:122) > at junit.framework.TestResult.runProtected(TestResult.java:142) > at junit.framework.TestResult.run(TestResult.java:125) > at junit.framework.TestCase.run(TestCase.java:129) > at junit.framework.TestSuite.runTest(TestSuite.java:252) > at junit.framework.TestSuite.run(TestSuite.java:247) > at > org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:86) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:105) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:56) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:64) > at > org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:50) > at sun.reflect.GeneratedMethodAccessor73.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32) > at > org.gradle.messaging.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93) > at com.sun.proxy.$Proxy2.processTestClass(Unknown Source) > at > org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:106) > at sun.reflect.GeneratedMethodAccessor72.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at >
[jira] [Commented] (GEODE-706) CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy
[ https://issues.apache.org/jira/browse/GEODE-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583552#comment-15583552 ] ASF subversion and git services commented on GEODE-706: --- Commit a3bd256648e30b0bf04c565a8f21d00868c29806 in incubator-geode's branch refs/heads/develop from [~hitesh.khamesra] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=a3bd256 ] GEODE-706 Fixed race condition between expiry thread and put thread. There was possibility that expiry thread destroying the entry and another thread doing update on same key. In this case expiry thread cancel's existing task and update thread adds expiry task. But this tasks are refer by regionEntry, which is same for both the threads. So in this case if expiry thread cancel's task after update thread then that entry will never expire. > CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy > - > > Key: GEODE-706 > URL: https://issues.apache.org/jira/browse/GEODE-706 > Project: Geode > Issue Type: Bug > Components: regions >Affects Versions: 1.0.0-incubating >Reporter: Jens Deppe >Assignee: Hitesh Khamesra > Labels: CI, Flaky > > git rev e3d24d77150382f363f334b3e2d6622c6e02e8bc build #948 > {noformat} > Error Message > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > Stacktrace > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > at com.gemstone.gemfire.internal.Assert.throwError(Assert.java:91) > at com.gemstone.gemfire.internal.Assert.assertTrue(Assert.java:115) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2074) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2034) > at > com.gemstone.gemfire.cache30.RegionTestCase.testEntryIdleDestroy(RegionTestCase.java:3569) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at junit.framework.TestCase.runTest(TestCase.java:176) > at junit.framework.TestCase.runBare(TestCase.java:141) > at junit.framework.TestResult$1.protect(TestResult.java:122) > at junit.framework.TestResult.runProtected(TestResult.java:142) > at junit.framework.TestResult.run(TestResult.java:125) > at junit.framework.TestCase.run(TestCase.java:129) > at junit.framework.TestSuite.runTest(TestSuite.java:252) > at junit.framework.TestSuite.run(TestSuite.java:247) > at > org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:86) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:105) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:56) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:64) > at > org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:50) > at sun.reflect.GeneratedMethodAccessor73.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32) > at > org.gradle.messaging.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93) > at com.sun.proxy.$Proxy2.processTestClass(Unknown Source) > at > org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:106) > at sun.reflect.GeneratedMethodAccessor72.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:360) > at >
[jira] [Commented] (GEODE-706) CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy
[ https://issues.apache.org/jira/browse/GEODE-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15245989#comment-15245989 ] Kirk Lund commented on GEODE-706: - time sensitive, expiration > CI failure: DiskDistributedNoAckAsyncRegionDUnitTest.testEntryIdleDestroy > - > > Key: GEODE-706 > URL: https://issues.apache.org/jira/browse/GEODE-706 > Project: Geode > Issue Type: Bug > Components: regions >Affects Versions: 1.0.0-incubating >Reporter: Jens Deppe > Labels: CI > > git rev e3d24d77150382f363f334b3e2d6622c6e02e8bc build #948 > {noformat} > Error Message > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > Stacktrace > com.gemstone.gemfire.InternalGemFireError: Entry failed to destroy > at com.gemstone.gemfire.internal.Assert.throwError(Assert.java:91) > at com.gemstone.gemfire.internal.Assert.assertTrue(Assert.java:115) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2074) > at > com.gemstone.gemfire.cache30.RegionTestCase.waitForDestroy(RegionTestCase.java:2034) > at > com.gemstone.gemfire.cache30.RegionTestCase.testEntryIdleDestroy(RegionTestCase.java:3569) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at junit.framework.TestCase.runTest(TestCase.java:176) > at junit.framework.TestCase.runBare(TestCase.java:141) > at junit.framework.TestResult$1.protect(TestResult.java:122) > at junit.framework.TestResult.runProtected(TestResult.java:142) > at junit.framework.TestResult.run(TestResult.java:125) > at junit.framework.TestCase.run(TestCase.java:129) > at junit.framework.TestSuite.runTest(TestSuite.java:252) > at junit.framework.TestSuite.run(TestSuite.java:247) > at > org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:86) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:105) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:56) > at > org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:64) > at > org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:50) > at sun.reflect.GeneratedMethodAccessor73.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32) > at > org.gradle.messaging.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93) > at com.sun.proxy.$Proxy2.processTestClass(Unknown Source) > at > org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:106) > at sun.reflect.GeneratedMethodAccessor72.invoke(Unknown Source) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:497) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35) > at > org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24) > at > org.gradle.messaging.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:360) > at > org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:54) > at > org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:40) > at > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) > at > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) > at java.lang.Thread.run(Thread.java:745) > Standard Output > Previously run tests: [IndexCommandsDUnitTest, ConfigCommandsDUnitTest, > FunctionCommandsDUnitTest, ListAndDescribeDiskStoreCommandsDUnitTest, > MiscellaneousCommandsExportLogsPart4DUnitTest, > MiscellaneousCommandsExportLogsPart2DUnitTest, >