Address review comments #951
46 passed, 8 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 1408s with 8 passed, 4 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.consumer.TestChangelogConsumer | 8✅ | 4❌ | 1408s |
❌ 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
14 tests were completed in 424s with 14 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication | 14✅ | 424s |
✅ com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication
✅ testActiveActiveForHeartbeatSystemStores
✅ testControllerBlocksConcurrentBatchPush
✅ testEmptyPush[0](false)
✅ testEmptyPush[2](true)
✅ testKilledRepushJobVersionStatus
✅ testNativeReplicationForBatchPush[0](50, 2)
✅ testNativeReplicationForBatchPush[2](1000, 10)
✅ testNativeReplicationForHybrid
✅ testNativeReplicationForIncrementalPush
✅ testNativeReplicationWithIngestionIsolationInDaVinci
✅ testNativeReplicationWithLeadershipHandover
✅ testPushDirectlyToChildRegion
✅ testTargetRegionPushWithDeferredVersionSwap
✅ testTargetedRegionPushJobFullConsumptionForBatchStore
✅ IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml
1 tests were completed in 80ms with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.zk.TestMigrateVeniceZKPaths | 1✅ | 80ms |
✅ 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 1408s with 8 passed, 4 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.consumer.TestChangelogConsumer | 8✅ | 4❌ | 1408s |
❌ 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
14 tests were completed in 424s with 14 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication | 14✅ | 424s |
✅ com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication
✅ testActiveActiveForHeartbeatSystemStores
✅ testControllerBlocksConcurrentBatchPush
✅ testEmptyPush[0](false)
✅ testEmptyPush[2](true)
✅ testKilledRepushJobVersionStatus
✅ testNativeReplicationForBatchPush[0](50, 2)
✅ testNativeReplicationForBatchPush[2](1000, 10)
✅ testNativeReplicationForHybrid
✅ testNativeReplicationForIncrementalPush
✅ testNativeReplicationWithIngestionIsolationInDaVinci
✅ testNativeReplicationWithLeadershipHandover
✅ testPushDirectlyToChildRegion
✅ testTargetRegionPushWithDeferredVersionSwap
✅ testTargetedRegionPushJobFullConsumptionForBatchStore
✅ internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml
1 tests were completed in 80ms with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.zk.TestMigrateVeniceZKPaths | 1✅ | 80ms |
✅ 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)