[controller] Add OTel metrics to DeferredVersionSwapStats #10442
VeniceCI-StaticAnalysisAndUnitTests.yml
on: pull_request
Matrix: Clients / UT & CodeCov
Matrix: Controller / UT & CodeCov
Matrix: Integrations / UT & CodeCov
Matrix: Internal / UT & CodeCov
Matrix: Router / UT & CodeCov
Matrix: Server / UT & CodeCov
Matrix: StaticAnalysis
ValidateGradleWrapper
14s
StaticAnalysisAndUnitTestsFailureAlert
Annotations
10 errors
|
SITWithPWiseAndBufferAfterLeaderTest.testRecordLevelMetricForCurrentVersion[0](false):
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseAndBufferAfterLeaderTest.java#L1
org.mockito.exceptions.verification.TooFewActualInvocations:
hostLevelIngestionStats.recordTotalBytesConsumed(
<any long>
);
Wanted 2 times:
-> at com.linkedin.davinci.stats.HostLevelIngestionStats.recordTotalBytesConsumed(HostLevelIngestionStats.java:523)
But was 1 time:
-> at com.linkedin.davinci.kafka.consumer.StoreIngestionTask.processConsumerRecord(StoreIngestionTask.java:2842)
|
|
SITWithPWiseAndBufferAfterLeaderTest.testRecordLevelMetricForCurrentVersion[0](false):
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseAndBufferAfterLeaderTest.java#L1
org.mockito.exceptions.verification.TooFewActualInvocations:
hostLevelIngestionStats.recordTotalBytesConsumed(
<any long>
);
Wanted 2 times:
-> at com.linkedin.davinci.stats.HostLevelIngestionStats.recordTotalBytesConsumed(HostLevelIngestionStats.java:523)
But was 1 time:
-> at com.linkedin.davinci.kafka.consumer.StoreIngestionTask.processConsumerRecord(StoreIngestionTask.java:2842)
|
|
InstanceSizeEstimatorTest.testInstanceMeasurement:
internal/venice-common/src/test/java/com/linkedin/venice/memory/InstanceSizeEstimatorTest.java#L49
java.lang.AssertionError: Memory allocated is negative! memoryAllocatedBeforeInstantiations: 2706537352; memoryAllocatedAfterInstantiations: 2663681416; memoryAllocatedByInstantiations: -42855936; 3 attempts left.
|
|
InstanceSizeEstimatorTest.testInstanceMeasurement:
internal/venice-common/src/test/java/com/linkedin/venice/memory/InstanceSizeEstimatorTest.java#L49
java.lang.AssertionError: Memory allocated is negative! memoryAllocatedBeforeInstantiations: 2706537352; memoryAllocatedAfterInstantiations: 2663681416; memoryAllocatedByInstantiations: -42855936; 3 attempts left.
|
|
NativeMetadataRepositoryTest.testNativeMetadataRepositoryStats:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/repository/NativeMetadataRepositoryTest.java#L149
java.lang.AssertionError: expected [2000.0] but found [1000.0]
|
|
SITWithPWiseWithoutBufferAfterLeaderTest.methodCleanUp:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseWithoutBufferAfterLeaderTest.java#L1
org.mockito.exceptions.misusing.CannotStubVoidMethodWithReturnValue:
'recordTotalAvgPartitionsPerConsumer' is a *void method* and it *cannot* be stubbed with a *return value*!
Voids are usually stubbed with Throwables:
doThrow(exception).when(mock).someVoidMethod();
If you need to set the void method to do nothing you can use:
doNothing().when(mock).someVoidMethod();
For more information, check out the javadocs for Mockito.doNothing().
***
If you're unsure why you're getting above error read on.
Due to the nature of the syntax above problem might occur because:
1. The method you are trying to stub is *overloaded*. Make sure you are calling the right overloaded version.
2. Somewhere in your test you are stubbing *final methods*. Sorry, Mockito does not verify/stub final methods.
3. A spy is stubbed using when(spy.foo()).then() syntax. It is safer to stub spies -
- with doReturn|Throw() family of methods. More in javadocs for Mockito.spy() method.
4. Mocking methods declared on non-public parent classes is not supported.
|
|
SITWithPWiseWithoutBufferAfterLeaderTest.methodSetUp:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseWithoutBufferAfterLeaderTest.java#L1
org.mockito.exceptions.misusing.CannotStubVoidMethodWithReturnValue:
'recordTotalPollRequestLatency' is a *void method* and it *cannot* be stubbed with a *return value*!
Voids are usually stubbed with Throwables:
doThrow(exception).when(mock).someVoidMethod();
If you need to set the void method to do nothing you can use:
doNothing().when(mock).someVoidMethod();
For more information, check out the javadocs for Mockito.doNothing().
***
If you're unsure why you're getting above error read on.
Due to the nature of the syntax above problem might occur because:
1. The method you are trying to stub is *overloaded*. Make sure you are calling the right overloaded version.
2. Somewhere in your test you are stubbing *final methods*. Sorry, Mockito does not verify/stub final methods.
3. A spy is stubbed using when(spy.foo()).then() syntax. It is safer to stub spies -
- with doReturn|Throw() family of methods. More in javadocs for Mockito.spy() method.
4. Mocking methods declared on non-public parent classes is not supported.
|
|
NativeMetadataRepositoryTest.testNativeMetadataRepositoryStats:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/repository/NativeMetadataRepositoryTest.java#L149
java.lang.AssertionError: expected [2000.0] but found [1000.0]
|
|
SITWithPWiseWithoutBufferAfterLeaderTest.methodCleanUp:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseWithoutBufferAfterLeaderTest.java#L1
org.mockito.exceptions.misusing.CannotStubVoidMethodWithReturnValue:
'recordTotalAvgPartitionsPerConsumer' is a *void method* and it *cannot* be stubbed with a *return value*!
Voids are usually stubbed with Throwables:
doThrow(exception).when(mock).someVoidMethod();
If you need to set the void method to do nothing you can use:
doNothing().when(mock).someVoidMethod();
For more information, check out the javadocs for Mockito.doNothing().
***
If you're unsure why you're getting above error read on.
Due to the nature of the syntax above problem might occur because:
1. The method you are trying to stub is *overloaded*. Make sure you are calling the right overloaded version.
2. Somewhere in your test you are stubbing *final methods*. Sorry, Mockito does not verify/stub final methods.
3. A spy is stubbed using when(spy.foo()).then() syntax. It is safer to stub spies -
- with doReturn|Throw() family of methods. More in javadocs for Mockito.spy() method.
4. Mocking methods declared on non-public parent classes is not supported.
|
|
SITWithPWiseWithoutBufferAfterLeaderTest.methodSetUp:
clients/da-vinci-client/src/test/java/com/linkedin/davinci/kafka/consumer/SITWithPWiseWithoutBufferAfterLeaderTest.java#L1
org.mockito.exceptions.misusing.CannotStubVoidMethodWithReturnValue:
'recordTotalPollRequestLatency' is a *void method* and it *cannot* be stubbed with a *return value*!
Voids are usually stubbed with Throwables:
doThrow(exception).when(mock).someVoidMethod();
If you need to set the void method to do nothing you can use:
doNothing().when(mock).someVoidMethod();
For more information, check out the javadocs for Mockito.doNothing().
***
If you're unsure why you're getting above error read on.
Due to the nature of the syntax above problem might occur because:
1. The method you are trying to stub is *overloaded*. Make sure you are calling the right overloaded version.
2. Somewhere in your test you are stubbing *final methods*. Sorry, Mockito does not verify/stub final methods.
3. A spy is stubbed using when(spy.foo()).then() syntax. It is safer to stub spies -
- with doReturn|Throw() family of methods. More in javadocs for Mockito.spy() method.
4. Mocking methods declared on non-public parent classes is not supported.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
StaticAnalysis
|
929 KB |
sha256:9de6fa6c14c6dbf0bdb9721086e2d48c288cd73624400ae7c0de537e725f3ef9
|
|
|
clients-jdk11
|
3.24 MB |
sha256:0b3a7599b275786f6326dfbc93f11923a2f76bcc857f009a20da284833780585
|
|
|
clients-jdk17
|
3.24 MB |
sha256:b92df49e7cb827428ec57a2d25e469052c16582caffdfbdd15105f49b6fe8fb8
|
|
|
clients-jdk8
|
3.2 MB |
sha256:16a60d8d1b2bad83d50507db96cca9915848c21abc246c90b29830bbd6c1b617
|
|
|
controller-jdk11
|
2.05 MB |
sha256:4e21102d481feb9db04ef29960c92fa47e8a03334c34cdc58f3bb80ea867f295
|
|
|
controller-jdk17
|
2.04 MB |
sha256:8a402c9807af9645601a27747fe050584099474c0e9e0a27f3b91039d4dba348
|
|
|
controller-jdk8
|
2.03 MB |
sha256:1c787e9f42f84864b931b01df59aa4477364408ef965feceb97a4c89bfd5beb2
|
|
|
integrations-jdk11
|
564 KB |
sha256:c4b86e59f3f80a89bdb8bd9cbf476352f02ea61d5e9f182b860000f06c68e6d1
|
|
|
integrations-jdk17
|
571 KB |
sha256:fb188d130080ae6648e4258f7f4dccbeb242c38fb8372db539333e0708bba5c3
|
|
|
integrations-jdk8
|
550 KB |
sha256:ba2b3e39ed533b3ea563caa642a2fd5218ccfbd2d169180dbf3aa5355e0d09f0
|
|
|
internal-jdk11
|
4.34 MB |
sha256:9e3c4494db45315e6635d8e38a95a40fe94edeb96cb40c1c2aa81a545620347a
|
|
|
internal-jdk17
|
4.35 MB |
sha256:b35f40c2d59d46d8b7c665b6c93d5b4525c1a758fbe759b47d5a9f7194823e3b
|
|
|
internal-jdk8
|
4.32 MB |
sha256:4b5ca3aa78f50eba7e8a791b0a17be67684ae34210ec48be7836857332cb929e
|
|
|
router-jdk11
|
770 KB |
sha256:537978be86454bb898a47e3d15b3be2688b069c2c7fa926f4c5be75e88c33140
|
|
|
router-jdk17
|
774 KB |
sha256:a7e59c244f8fd96eabe1d2b95976b95cc2b36a19822ce13bd7f842c3210b680a
|
|
|
router-jdk8
|
755 KB |
sha256:8448d6cd0369b5ad1782ef37d1e8ef2210b8fbc3935becff8a039180b3663f6b
|
|
|
server-jdk11
|
10.9 MB |
sha256:3cb93b74a743f5dbd1cbf3458387226ce0a726a8a7da0c2ada71587192f25717
|
|
|
server-jdk17
|
12.1 MB |
sha256:8f50fda67b6c4e373ce121d6109fc1b598085342e0f2e94a8e5d3c07a3e0a415
|
|
|
server-jdk8
|
10.8 MB |
sha256:e5a90021a3bfc63af95a5589e7904964e8d012b6e0e1a001f89abd2bb8434c34
|
|