Skip to content

Address review comments #951

Address review comments

Address review comments #951

GitHub Actions / IntegrationTests_9 Test Reports succeeded Nov 25, 2025 in 0s

84 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment.xml 9✅ 162s
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.PushJobDetailsTest.xml 4✅ 397s
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions.xml 2✅ 161s
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestStoreMigration.xml 11✅ 366s
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest.xml 8✅ 3s
IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.helix.TestHelixExternalViewRepository.xml 8✅ 51s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment.xml 9✅ 162s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.PushJobDetailsTest.xml 4✅ 397s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions.xml 2✅ 161s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestStoreMigration.xml 11✅ 366s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest.xml 8✅ 3s
internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.helix.TestHelixExternalViewRepository.xml 8✅ 51s

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment 9✅ 162s

✅ com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment

✅ testAbortMigrationStoreDeletion
✅ testControllerFailOver
✅ testDeadStoreStatsInitialization
✅ testExternalViewDataChangeDeadLock
✅ testGetFutureVersionsNotBlocked
✅ testGetLeaderController
✅ testIdempotentStoreDeletion
✅ testStoreLifecycleValidateDeleted_WithLeftoverTopics
✅ testStoreLifecycle_ValidateDeleted

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.PushJobDetailsTest.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.PushJobDetailsTest 4✅ 397s

✅ com.linkedin.venice.endToEnd.PushJobDetailsTest

✅ testPushJobDetails[0](false, false)
✅ testPushJobDetails[2](true, false)
✅ testPushJobDetails[4](false, true)
✅ testPushJobDetails[6](true, true)

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions 2✅ 161s

✅ com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions

✅ testDeferredVersionSwapWithFailedPushInTargetRegions
✅ testDvcDelayedIngestionWithFailingPushInTargetRegion

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestStoreMigration.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestStoreMigration 11✅ 366s

✅ com.linkedin.venice.endToEnd.TestStoreMigration

✅ testAutoStoreMigration
✅ testStoreMigration
✅ testStoreMigrationAfterFailedAttempt
✅ testStoreMigrationForFastClient
✅ testStoreMigrationStaleKillIngestionMessageDeletion
✅ testStoreMigrationWithDaVinciPushStatusSystemStore
✅ testStoreMigrationWithMetaSystemStore
✅ testStoreMigrationWithNewPushesAndUpdates
✅ testStoreMigrationWithPushJobTrackingVTInParentRegion
✅ testStoreMigrationWithStoreConfigs
✅ testVersionConfigsRemainSameAfterStoreMigration

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest 8✅ 3s

✅ com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest

✅ testBatchGetGenericClient[0](false, SERVER_BASED_METADATA)
✅ testBatchGetGenericClient[2](true, SERVER_BASED_METADATA)
✅ testBatchGetSpecificClient[0](SERVER_BASED_METADATA)
✅ testStreamingBatchGetGenericClient[0](false, SERVER_BASED_METADATA)
✅ testStreamingBatchGetGenericClient[2](true, SERVER_BASED_METADATA)
✅ testStreamingBatchGetLimit
✅ testStreamingBatchGetWithCallbackGenericClient[0](false, SERVER_BASED_METADATA)
✅ testStreamingBatchGetWithCallbackGenericClient[2](true, SERVER_BASED_METADATA)

✅ IntegrationTests_9-artifacts/internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.helix.TestHelixExternalViewRepository.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.helix.TestHelixExternalViewRepository 8✅ 51s

✅ com.linkedin.venice.helix.TestHelixExternalViewRepository

✅ testControllerChanged
✅ testGetInstances
✅ testGetNumberOfPartitions
✅ testGetNumberOfPartitionsWhenResourceDropped
✅ testGetPartitions
✅ testListeners
✅ testNodeChanged
✅ testPartitionMove

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment 9✅ 162s

✅ com.linkedin.venice.controller.TestVeniceHelixAdminWithIsolatedEnvironment

✅ testAbortMigrationStoreDeletion
✅ testControllerFailOver
✅ testDeadStoreStatsInitialization
✅ testExternalViewDataChangeDeadLock
✅ testGetFutureVersionsNotBlocked
✅ testGetLeaderController
✅ testIdempotentStoreDeletion
✅ testStoreLifecycleValidateDeleted_WithLeftoverTopics
✅ testStoreLifecycle_ValidateDeleted

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.PushJobDetailsTest.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.PushJobDetailsTest 4✅ 397s

✅ com.linkedin.venice.endToEnd.PushJobDetailsTest

✅ testPushJobDetails[0](false, false)
✅ testPushJobDetails[2](true, false)
✅ testPushJobDetails[4](false, true)
✅ testPushJobDetails[6](true, true)

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions 2✅ 161s

✅ com.linkedin.venice.endToEnd.TestDeferredVersionSwapWithFailingRegions

✅ testDeferredVersionSwapWithFailedPushInTargetRegions
✅ testDvcDelayedIngestionWithFailingPushInTargetRegion

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.endToEnd.TestStoreMigration.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.endToEnd.TestStoreMigration 11✅ 366s

✅ com.linkedin.venice.endToEnd.TestStoreMigration

✅ testAutoStoreMigration
✅ testStoreMigration
✅ testStoreMigrationAfterFailedAttempt
✅ testStoreMigrationForFastClient
✅ testStoreMigrationStaleKillIngestionMessageDeletion
✅ testStoreMigrationWithDaVinciPushStatusSystemStore
✅ testStoreMigrationWithMetaSystemStore
✅ testStoreMigrationWithNewPushesAndUpdates
✅ testStoreMigrationWithPushJobTrackingVTInParentRegion
✅ testStoreMigrationWithStoreConfigs
✅ testVersionConfigsRemainSameAfterStoreMigration

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest 8✅ 3s

✅ com.linkedin.venice.fastclient.BatchGetAvroStoreClientTest

✅ testBatchGetGenericClient[0](false, SERVER_BASED_METADATA)
✅ testBatchGetGenericClient[2](true, SERVER_BASED_METADATA)
✅ testBatchGetSpecificClient[0](SERVER_BASED_METADATA)
✅ testStreamingBatchGetGenericClient[0](false, SERVER_BASED_METADATA)
✅ testStreamingBatchGetGenericClient[2](true, SERVER_BASED_METADATA)
✅ testStreamingBatchGetLimit
✅ testStreamingBatchGetWithCallbackGenericClient[0](false, SERVER_BASED_METADATA)
✅ testStreamingBatchGetWithCallbackGenericClient[2](true, SERVER_BASED_METADATA)

✅ internal/venice-test-common/build/test-results/integrationTests_9/TEST-com.linkedin.venice.helix.TestHelixExternalViewRepository.xml

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

Test suite Passed Failed Skipped Time
com.linkedin.venice.helix.TestHelixExternalViewRepository 8✅ 51s

✅ com.linkedin.venice.helix.TestHelixExternalViewRepository

✅ testControllerChanged
✅ testGetInstances
✅ testGetNumberOfPartitions
✅ testGetNumberOfPartitionsWhenResourceDropped
✅ testGetPartitions
✅ testListeners
✅ testNodeChanged
✅ testPartitionMove