Skip to content

Address review comments #951

Address review comments

Address review comments #951

GitHub Actions / IntegrationTests_7 Test Reports succeeded Nov 24, 2025 in 0s

20 passed, 8 failed and 0 skipped

Tests failed

Report Passed Failed Skipped Time
IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml 8✅ 4❌ 948s
IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.kafka.KafkaConsumptionTest.xml 2✅ 2s
internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml 8✅ 4❌ 948s
internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.kafka.KafkaConsumptionTest.xml 2✅ 2s

❌ IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml

12 tests were completed in 948s with 8 passed, 4 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing 8✅ 4❌ 948s

❌ com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing

✅ 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_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.kafka.KafkaConsumptionTest.xml

2 tests were completed in 2s with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.kafka.KafkaConsumptionTest 2✅ 2s

✅ com.linkedin.venice.kafka.KafkaConsumptionTest

✅ testLocalAndRemoteConsumption[0](PARTITION_WISE_SHARED_CONSUMER_ASSIGNMENT_STRATEGY)
✅ testLocalAndRemoteConsumption[2](STORE_AWARE_PARTITION_WISE_SHARED_CONSUMER_ASSIGNMENT_STRATEGY)

❌ internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml

12 tests were completed in 948s with 8 passed, 4 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing 8✅ 4❌ 948s

❌ com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing

✅ 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_7/TEST-com.linkedin.venice.kafka.KafkaConsumptionTest.xml

2 tests were completed in 2s with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.linkedin.venice.kafka.KafkaConsumptionTest 2✅ 2s

✅ com.linkedin.venice.kafka.KafkaConsumptionTest

✅ testLocalAndRemoteConsumption[0](PARTITION_WISE_SHARED_CONSUMER_ASSIGNMENT_STRATEGY)
✅ testLocalAndRemoteConsumption[2](STORE_AWARE_PARTITION_WISE_SHARED_CONSUMER_ASSIGNMENT_STRATEGY)

Annotations

Check failure on line 891 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_7 Test Reports

com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing ► testSpecificRecordVeniceChangelogConsumer

Failed test found in:
  IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml
  internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.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)

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_7 Test Reports

com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing ► testVersionSpecificChangeLogConsumer

Failed test found in:
  IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml
  internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.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_7 Test Reports

com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing ► testVersionSpecificSeekingChangeLogConsumer

Failed test found in:
  IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml
  IntegrationTests_7-artifacts/internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml
  internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.xml
  internal/venice-test-common/build/test-results/integrationTests_7/TEST-com.linkedin.venice.consumer.TestChangelogConsumerWithParallelProcessing.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)