Address review comments #943
46 passed, 10 failed and 0 skipped
❌ IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
12 tests were completed in 1519s with 8 passed, 4 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.consumer.TestChangelogConsumer | 8✅ | 4❌ | 1519s |
❌ com.linkedin.venice.consumer.TestChangelogConsumer
✅ testAAIngestionWithStoreView
✅ testChangeLogConsumerSequenceId
✅ testChangelogConsumerWithNewValueSchema
✅ testNewChangelogConsumerWithNewValueSchema
❌ testSpecificRecordVeniceChangelogConsumer
java.lang.AssertionError: expected [2] but found [1]
❌ testVersionSpecificChangeLogConsumer
java.lang.AssertionError: expected [100] but found [0]
❌ testVersionSpecificSeekingChangeLogConsumer
java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSwapInALoop
✅ testSpecificRecordVeniceChangelogConsumer
✅ testVersionSpecificChangeLogConsumer
❌ testVersionSpecificSeekingChangeLogConsumer
java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSpecificSeekingChangeLogConsumer
❌ IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml
15 tests were completed in 724s with 14 passed, 1 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication | 14✅ | 1❌ | 724s |
❌ com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication
✅ testActiveActiveForHeartbeatSystemStores
✅ testControllerBlocksConcurrentBatchPush
✅ testEmptyPush[0](false)
✅ testEmptyPush[2](true)
❌ testKilledRepushJobVersionStatus
java.lang.AssertionError: java.util.NoSuchElementException: No value present
✅ testNativeReplicationForBatchPush[0](50, 2)
✅ testNativeReplicationForBatchPush[2](1000, 10)
✅ testNativeReplicationForHybrid
✅ testNativeReplicationForIncrementalPush
✅ testNativeReplicationWithIngestionIsolationInDaVinci
✅ testNativeReplicationWithLeadershipHandover
✅ testPushDirectlyToChildRegion
✅ testTargetRegionPushWithDeferredVersionSwap
✅ testTargetedRegionPushJobFullConsumptionForBatchStore
✅ testKilledRepushJobVersionStatus
✅ IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml
1 tests were completed in 84ms with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.zk.TestMigrateVeniceZKPaths | 1✅ | 84ms |
✅ com.linkedin.venice.zk.TestMigrateVeniceZKPaths
✅ testMigrateVenicePaths
❌ internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
12 tests were completed in 1519s with 8 passed, 4 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.consumer.TestChangelogConsumer | 8✅ | 4❌ | 1519s |
❌ com.linkedin.venice.consumer.TestChangelogConsumer
✅ testAAIngestionWithStoreView
✅ testChangeLogConsumerSequenceId
✅ testChangelogConsumerWithNewValueSchema
✅ testNewChangelogConsumerWithNewValueSchema
❌ testSpecificRecordVeniceChangelogConsumer
java.lang.AssertionError: expected [2] but found [1]
❌ testVersionSpecificChangeLogConsumer
java.lang.AssertionError: expected [100] but found [0]
❌ testVersionSpecificSeekingChangeLogConsumer
java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSwapInALoop
✅ testSpecificRecordVeniceChangelogConsumer
✅ testVersionSpecificChangeLogConsumer
❌ testVersionSpecificSeekingChangeLogConsumer
java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSpecificSeekingChangeLogConsumer
❌ internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml
15 tests were completed in 724s with 14 passed, 1 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication | 14✅ | 1❌ | 724s |
❌ com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication
✅ testActiveActiveForHeartbeatSystemStores
✅ testControllerBlocksConcurrentBatchPush
✅ testEmptyPush[0](false)
✅ testEmptyPush[2](true)
❌ testKilledRepushJobVersionStatus
java.lang.AssertionError: java.util.NoSuchElementException: No value present
✅ testNativeReplicationForBatchPush[0](50, 2)
✅ testNativeReplicationForBatchPush[2](1000, 10)
✅ testNativeReplicationForHybrid
✅ testNativeReplicationForIncrementalPush
✅ testNativeReplicationWithIngestionIsolationInDaVinci
✅ testNativeReplicationWithLeadershipHandover
✅ testPushDirectlyToChildRegion
✅ testTargetRegionPushWithDeferredVersionSwap
✅ testTargetedRegionPushJobFullConsumptionForBatchStore
✅ testKilledRepushJobVersionStatus
✅ internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml
1 tests were completed in 84ms with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.zk.TestMigrateVeniceZKPaths | 1✅ | 84ms |
✅ com.linkedin.venice.zk.TestMigrateVeniceZKPaths
✅ testMigrateVenicePaths
Annotations
github-actions / IntegrationTests_8 Test Reports
com.linkedin.venice.consumer.TestChangelogConsumer ► testSpecificRecordVeniceChangelogConsumer
Failed test found in:
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
Error:
java.lang.AssertionError: expected [2] but found [1]
Raw output
java.lang.AssertionError: expected [2] but found [1]
at org.testng.Assert.fail(Assert.java:96)
at org.testng.Assert.failNotEquals(Assert.java:776)
at org.testng.Assert.assertEqualsImpl(Assert.java:137)
at org.testng.Assert.assertEquals(Assert.java:118)
at org.testng.Assert.assertEquals(Assert.java:652)
at org.testng.Assert.assertEquals(Assert.java:662)
at com.linkedin.venice.consumer.TestChangelogConsumer.lambda$testSpecificRecordVeniceChangelogConsumer$44(TestChangelogConsumer.java:891)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:287)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:272)
at com.linkedin.venice.consumer.TestChangelogConsumer.testSpecificRecordVeniceChangelogConsumer(TestChangelogConsumer.java:886)
github-actions / IntegrationTests_8 Test Reports
com.linkedin.venice.consumer.TestChangelogConsumer ► testVersionSpecificChangeLogConsumer
Failed test found in:
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
Error:
java.lang.AssertionError: expected [100] but found [0]
Raw output
java.lang.AssertionError: expected [100] but found [0]
at org.testng.Assert.fail(Assert.java:96)
at org.testng.Assert.failNotEquals(Assert.java:776)
at org.testng.Assert.assertEqualsImpl(Assert.java:137)
at org.testng.Assert.assertEquals(Assert.java:118)
at org.testng.Assert.assertEquals(Assert.java:652)
at org.testng.Assert.assertEquals(Assert.java:662)
at com.linkedin.venice.consumer.TestChangelogConsumer.lambda$testVersionSpecificChangeLogConsumer$57(TestChangelogConsumer.java:1325)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:287)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:272)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:264)
at com.linkedin.venice.consumer.TestChangelogConsumer.testVersionSpecificChangeLogConsumer(TestChangelogConsumer.java:1319)
github-actions / IntegrationTests_8 Test Reports
com.linkedin.venice.consumer.TestChangelogConsumer ► testVersionSpecificSeekingChangeLogConsumer
Failed test found in:
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml
Error:
java.lang.AssertionError: expected [1] but found [0]
Raw output
java.lang.AssertionError: expected [1] but found [0]
at org.testng.Assert.fail(Assert.java:96)
at org.testng.Assert.failNotEquals(Assert.java:776)
at org.testng.Assert.assertEqualsImpl(Assert.java:137)
at org.testng.Assert.assertEquals(Assert.java:118)
at org.testng.Assert.assertEquals(Assert.java:652)
at org.testng.Assert.assertEquals(Assert.java:662)
at com.linkedin.venice.consumer.TestChangelogConsumer.lambda$testVersionSpecificSeekingChangeLogConsumer$55(TestChangelogConsumer.java:1230)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:287)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:272)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:264)
at com.linkedin.venice.consumer.TestChangelogConsumer.testVersionSpecificSeekingChangeLogConsumer(TestChangelogConsumer.java:1227)
github-actions / IntegrationTests_8 Test Reports
com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication ► testKilledRepushJobVersionStatus
Failed test found in:
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml
Error:
java.lang.AssertionError: java.util.NoSuchElementException: No value present
Raw output
java.lang.AssertionError: java.util.NoSuchElementException: No value present
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:292)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:272)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:264)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.lambda$testKilledRepushJobVersionStatus$40(TestPushJobWithNativeReplication.java:859)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.motherOfAllTests(TestPushJobWithNativeReplication.java:930)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.testKilledRepushJobVersionStatus(TestPushJobWithNativeReplication.java:807)
Caused by: java.util.NoSuchElementException: No value present
at java.base/java.util.Optional.get(Optional.java:143)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.lambda$testKilledRepushJobVersionStatus$38(TestPushJobWithNativeReplication.java:863)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:287)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:272)
at com.linkedin.venice.utils.TestUtils.waitForNonDeterministicAssertion(TestUtils.java:264)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.lambda$testKilledRepushJobVersionStatus$40(TestPushJobWithNativeReplication.java:859)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.motherOfAllTests(TestPushJobWithNativeReplication.java:930)
at com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.testKilledRepushJobVersionStatus(TestPushJobWithNativeReplication.java:807)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:569)
at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:124)
at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:54)
at org.testng.internal.InvokeMethodRunnable.run(InvokeMethodRunnable.java:44)
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
at java.base/java.lang.Thread.run(Thread.java:840)