Address review comments #946
158 passed, 0 failed and 0 skipped
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter.xml
5 tests were completed in 125s with 5 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter | 5✅ | 125s |
✅ com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter
✅ testCreateStore
✅ testOldStoresWithHybridStoreVersioning
✅ testEmptyPush
✅ testPartitionCountUpdate
✅ testDeleteAndRecreateStore
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.controllerapi.TestControllerClient.xml
6 tests were completed in 153s with 6 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.controllerapi.TestControllerClient | 6✅ | 153s |
✅ com.linkedin.venice.controllerapi.TestControllerClient
✅ clientReturnsErrorObjectOnConnectionFailure
✅ testControllerClientWithInvalidUrls
✅ testD2ControllerClient
✅ testD2ControllerClientWithExternalD2Client
✅ testGetLeaderControllerUrlWithUrlContainingSpace
✅ testObjectMapperIgnoringUnknownProperties
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest.xml
2 tests were completed in 104s with 2 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest | 2✅ | 104s |
✅ com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest
✅ testDaVinciVersionSwap
✅ testMultiClusterDaVinci
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.MetaSystemStoreTest.xml
4 tests were completed in 105s with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.MetaSystemStoreTest | 4✅ | 105s |
✅ com.linkedin.venice.endToEnd.MetaSystemStoreTest
✅ bootstrapMetaSystemStore
✅ testParentControllerAutoMaterializeMetaSystemStore
✅ testThinClientMetaStoreBasedRepository
✅ testThinClientMetaStoreBasedRepositoryWithLargeValueSchemas
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush.xml
3 tests were completed in 111s with 3 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush | 3✅ | 111s |
✅ com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush
✅ testAAReplicationForIncPush
✅ testAAReplicationForIncPushWithSeparateRT
✅ testRealTimeTopicPartitionCountMatchesHybridVersion
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion.xml
1 tests were completed in 94s with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion | 1✅ | 94s |
✅ com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion
✅ testStoreMigrationMultiRegion
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.helix.StoragePersonaRepositoryTest.xml
25 tests were completed in 5s with 25 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.helix.StoragePersonaRepositoryTest | 25✅ | 5s |
✅ com.linkedin.venice.helix.StoragePersonaRepositoryTest
✅ testAddPersonaPersistence
✅ testAddPersonaThenMutate
✅ testDeletePersona
✅ testDeletePersonaDoesNotExist
✅ testDeletePersonaReUseName
✅ testGetAllPersonas
✅ testGetAllPersonasDeletePersonas
✅ testGetAllPersonasNoPersonaExists
✅ testGetPersona
✅ testPersonaNameExists
✅ testUpdateQuotaAndStoresFailedDoesNotExist
✅ testUpdateQuotaFailed
✅ testUpdateQuotaFailedDoesNotExist
✅ testUpdateQuotaSuccess
✅ testUpdateRemoveStores
✅ testUpdateStoresAlreadyContainedByPersona
✅ testUpdateStoresAndQuotaPersistence
✅ testUpdateStoresFailInvalidStore
✅ testUpdateStoresFailedDoesNotExist
✅ testUpdateStoresFailedInvalidQuota
✅ testUpdateStoresSuccess
✅ testValidatePersonaNoOwners
✅ testValidatePersonaStoreDoesNotExist
✅ testValidatePersonaStoreQuotaInvalid
✅ testValidateStoreAlreadyContainedByPersona
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest.xml
4 tests were completed in 773ms with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest | 4✅ | 773ms |
✅ com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest
✅ testProducerCloseCanUnblockSendMessageCallerThread[0](false)
✅ testProducerCloseCanUnblockSendMessageCallerThread[2](true)
✅ testProducerCloseDoesNotLeaveAnyFuturesIncomplete[0](false)
✅ testProducerCloseDoesNotLeaveAnyFuturesIncomplete[2](true)
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest.xml
25 tests were completed in 153s with 25 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest | 25✅ | 153s |
✅ com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest
✅ testBatchUnsubscribeForExistingAndNonExistentTopic
✅ testBeginningOffsetForNonExistentTopic
✅ testBeginningPositionForExistingTopic
✅ testBeginningPositionForExistingTopicWithNonExistentPartition
✅ testEndPositionWithoutExplicitTimeoutForNonExistentPartition
✅ testEndPositionsForExistingTopic
✅ testEndPositionsForExistingTopicWithNonExistentPartition
✅ testEndPositionsForNonExistentAndExistingTopic
✅ testEndPositionsForNonExistentTopic
✅ testGetPositionByTimestampForExistingTopicWithInvalidPartition
✅ testGetPositionByTimestampForExistingTopicWithValidPartitionsAndMessages
✅ testGetPositionByTimestampForExistingTopicWithValidPartitionsButNoMessages
✅ testGetPositionByTimestampForNonExistentTopic
✅ testGetPositionByTimestampWithoutExplicitTimeoutForExistingTopicWithInvalidPartition
✅ testGetPositionByTimestampWithoutExplicitTimeoutForExistingTopicWithValidPartitionsButNoMessages
✅ testGetPositionByTimestampWithoutExplicitTimeoutForNonExistentTopic
✅ testPartitionsForExistingTopic
✅ testPartitionsForNonExistentTopic
✅ testPollPauseResume
✅ testResetOffsetForExistingTopicWithValidPartitionAndSubscription
✅ testResetOffsetForExistingTopicWithValidPartitionButNoSubscription
✅ testSubscribeForExistingTopicWithExistingPartition
✅ testSubscribeForExistingTopicWithNonExistentPartition
✅ testSubscribeForNonExistentTopic
✅ testUnsubscribeForExistingAndNonExistentTopic
✅ IntegrationTests_15-artifacts/internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion.xml
4 tests were completed in 71s with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion | 4✅ | 71s |
✅ com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion
✅ testActiveActiveStoreWithRMDAndRestartServer[0](false, false)
✅ testActiveActiveStoreWithRMDAndRestartServer[2](true, false)
✅ testActiveActiveStoreWithRMDAndRestartServer[4](false, true)
✅ testActiveActiveStoreWithRMDAndRestartServer[6](true, true)
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter.xml
5 tests were completed in 125s with 5 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter | 5✅ | 125s |
✅ com.linkedin.venice.controller.TestHybridStoreRepartitioningWithMultiDataCenter
✅ testCreateStore
✅ testOldStoresWithHybridStoreVersioning
✅ testEmptyPush
✅ testPartitionCountUpdate
✅ testDeleteAndRecreateStore
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.controllerapi.TestControllerClient.xml
6 tests were completed in 153s with 6 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.controllerapi.TestControllerClient | 6✅ | 153s |
✅ com.linkedin.venice.controllerapi.TestControllerClient
✅ clientReturnsErrorObjectOnConnectionFailure
✅ testControllerClientWithInvalidUrls
✅ testD2ControllerClient
✅ testD2ControllerClientWithExternalD2Client
✅ testGetLeaderControllerUrlWithUrlContainingSpace
✅ testObjectMapperIgnoringUnknownProperties
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest.xml
2 tests were completed in 104s with 2 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest | 2✅ | 104s |
✅ com.linkedin.venice.endToEnd.DaVinciClusterAgnosticTest
✅ testDaVinciVersionSwap
✅ testMultiClusterDaVinci
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.MetaSystemStoreTest.xml
4 tests were completed in 105s with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.MetaSystemStoreTest | 4✅ | 105s |
✅ com.linkedin.venice.endToEnd.MetaSystemStoreTest
✅ bootstrapMetaSystemStore
✅ testParentControllerAutoMaterializeMetaSystemStore
✅ testThinClientMetaStoreBasedRepository
✅ testThinClientMetaStoreBasedRepositoryWithLargeValueSchemas
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush.xml
3 tests were completed in 111s with 3 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush | 3✅ | 111s |
✅ com.linkedin.venice.endToEnd.TestActiveActiveReplicationForIncPush
✅ testAAReplicationForIncPush
✅ testAAReplicationForIncPushWithSeparateRT
✅ testRealTimeTopicPartitionCountMatchesHybridVersion
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion.xml
1 tests were completed in 94s with 1 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion | 1✅ | 94s |
✅ com.linkedin.venice.endToEnd.TestStoreMigrationMultiRegion
✅ testStoreMigrationMultiRegion
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.helix.StoragePersonaRepositoryTest.xml
25 tests were completed in 5s with 25 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.helix.StoragePersonaRepositoryTest | 25✅ | 5s |
✅ com.linkedin.venice.helix.StoragePersonaRepositoryTest
✅ testAddPersonaPersistence
✅ testAddPersonaThenMutate
✅ testDeletePersona
✅ testDeletePersonaDoesNotExist
✅ testDeletePersonaReUseName
✅ testGetAllPersonas
✅ testGetAllPersonasDeletePersonas
✅ testGetAllPersonasNoPersonaExists
✅ testGetPersona
✅ testPersonaNameExists
✅ testUpdateQuotaAndStoresFailedDoesNotExist
✅ testUpdateQuotaFailed
✅ testUpdateQuotaFailedDoesNotExist
✅ testUpdateQuotaSuccess
✅ testUpdateRemoveStores
✅ testUpdateStoresAlreadyContainedByPersona
✅ testUpdateStoresAndQuotaPersistence
✅ testUpdateStoresFailInvalidStore
✅ testUpdateStoresFailedDoesNotExist
✅ testUpdateStoresFailedInvalidQuota
✅ testUpdateStoresSuccess
✅ testValidatePersonaNoOwners
✅ testValidatePersonaStoreDoesNotExist
✅ testValidatePersonaStoreQuotaInvalid
✅ testValidateStoreAlreadyContainedByPersona
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest.xml
4 tests were completed in 773ms with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest | 4✅ | 773ms |
✅ com.linkedin.venice.pubsub.adapter.kafka.producer.ApacheKafkaProducerAdapterITest
✅ testProducerCloseCanUnblockSendMessageCallerThread[0](false)
✅ testProducerCloseCanUnblockSendMessageCallerThread[2](true)
✅ testProducerCloseDoesNotLeaveAnyFuturesIncomplete[0](false)
✅ testProducerCloseDoesNotLeaveAnyFuturesIncomplete[2](true)
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest.xml
25 tests were completed in 153s with 25 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest | 25✅ | 153s |
✅ com.linkedin.venice.pubsub.api.consumer.PubSubConsumerAdapterTest
✅ testBatchUnsubscribeForExistingAndNonExistentTopic
✅ testBeginningOffsetForNonExistentTopic
✅ testBeginningPositionForExistingTopic
✅ testBeginningPositionForExistingTopicWithNonExistentPartition
✅ testEndPositionWithoutExplicitTimeoutForNonExistentPartition
✅ testEndPositionsForExistingTopic
✅ testEndPositionsForExistingTopicWithNonExistentPartition
✅ testEndPositionsForNonExistentAndExistingTopic
✅ testEndPositionsForNonExistentTopic
✅ testGetPositionByTimestampForExistingTopicWithInvalidPartition
✅ testGetPositionByTimestampForExistingTopicWithValidPartitionsAndMessages
✅ testGetPositionByTimestampForExistingTopicWithValidPartitionsButNoMessages
✅ testGetPositionByTimestampForNonExistentTopic
✅ testGetPositionByTimestampWithoutExplicitTimeoutForExistingTopicWithInvalidPartition
✅ testGetPositionByTimestampWithoutExplicitTimeoutForExistingTopicWithValidPartitionsButNoMessages
✅ testGetPositionByTimestampWithoutExplicitTimeoutForNonExistentTopic
✅ testPartitionsForExistingTopic
✅ testPartitionsForNonExistentTopic
✅ testPollPauseResume
✅ testResetOffsetForExistingTopicWithValidPartitionAndSubscription
✅ testResetOffsetForExistingTopicWithValidPartitionButNoSubscription
✅ testSubscribeForExistingTopicWithExistingPartition
✅ testSubscribeForExistingTopicWithNonExistentPartition
✅ testSubscribeForNonExistentTopic
✅ testUnsubscribeForExistingAndNonExistentTopic
✅ internal/venice-test-common/build/test-results/integrationTests_15/TEST-com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion.xml
4 tests were completed in 71s with 4 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion | 4✅ | 71s |
✅ com.linkedin.venice.restart.TestRestartServerAfterDeletingSstFilesWithActiveActiveIngestion
✅ testActiveActiveStoreWithRMDAndRestartServer[0](false, false)
✅ testActiveActiveStoreWithRMDAndRestartServer[2](true, false)
✅ testActiveActiveStoreWithRMDAndRestartServer[4](false, true)
✅ testActiveActiveStoreWithRMDAndRestartServer[6](true, true)