Skip to content

Address review comments #946

Address review comments

Address review comments #946

GitHub Actions / IntegrationTests_8 Test Reports succeeded Nov 22, 2025 in 1s

46 passed, 6 failed and 0 skipped

Tests failed

Report Passed Failed Skipped Time
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml 8✅ 3❌ 1316s
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml 14✅ 419s
IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml 1✅ 82ms
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml 8✅ 3❌ 1316s
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication.xml 14✅ 419s
internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.zk.TestMigrateVeniceZKPaths.xml 1✅ 82ms

❌ IntegrationTests_8-artifacts/internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml

11 tests were completed in 1316s with 8 passed, 3 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.consumer.TestChangelogConsumer 8✅ 3❌ 1316s

❌ com.linkedin.venice.consumer.TestChangelogConsumer

✅ testAAIngestionWithStoreView
✅ testChangeLogConsumerSequenceId
✅ testChangelogConsumerWithNewValueSchema
✅ testNewChangelogConsumerWithNewValueSchema
✅ testSpecificRecordVeniceChangelogConsumer
❌ testVersionSpecificChangeLogConsumer
	java.lang.AssertionError: expected [100] but found [0]
❌ testVersionSpecificSeekingChangeLogConsumer
	java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSwapInALoop
✅ 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 419s with 14 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication 14✅ 419s

✅ 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 82ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.zk.TestMigrateVeniceZKPaths 1✅ 82ms

✅ com.linkedin.venice.zk.TestMigrateVeniceZKPaths

✅ testMigrateVenicePaths

❌ internal/venice-test-common/build/test-results/integrationTests_8/TEST-com.linkedin.venice.consumer.TestChangelogConsumer.xml

11 tests were completed in 1316s with 8 passed, 3 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.consumer.TestChangelogConsumer 8✅ 3❌ 1316s

❌ com.linkedin.venice.consumer.TestChangelogConsumer

✅ testAAIngestionWithStoreView
✅ testChangeLogConsumerSequenceId
✅ testChangelogConsumerWithNewValueSchema
✅ testNewChangelogConsumerWithNewValueSchema
✅ testSpecificRecordVeniceChangelogConsumer
❌ testVersionSpecificChangeLogConsumer
	java.lang.AssertionError: expected [100] but found [0]
❌ testVersionSpecificSeekingChangeLogConsumer
	java.lang.AssertionError: expected [1] but found [0]
✅ testVersionSwapInALoop
✅ 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 419s with 14 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestPushJobWithNativeReplication 14✅ 419s

✅ 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 82ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.zk.TestMigrateVeniceZKPaths 1✅ 82ms

✅ com.linkedin.venice.zk.TestMigrateVeniceZKPaths

✅ testMigrateVenicePaths

Annotations

Check failure on line 1325 in internal/venice-test-common/src/integrationTest/java/com/linkedin/venice/consumer/TestChangelogConsumer.java

See this annotation in the file changed.

@github-actions 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)

Check failure on line 1230 in internal/venice-test-common/src/integrationTest/java/com/linkedin/venice/consumer/TestChangelogConsumer.java

See this annotation in the file changed.

@github-actions 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)