-
Notifications
You must be signed in to change notification settings - Fork 524
Open Telemetry: Adds option to opt-in optional dimensions #4925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open Telemetry: Adds option to opt-in optional dimensions #4925
Conversation
more changes wip rewrite meters refactor code dimension names test fix contract update import fixes wip added other metrics code refactor add docuemnattion Delete Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.EmulatorTests/Microsoft - Backup.Azure.Cosmos.EmulatorTests.csproj upodated contract
…ripts (#4813) [INTERNAL] CI: Fixes emulator set-up to leverage central SDK teams scripts Observations - Lessmsi: extraction taking longer (1M) then msiexec based model - Starting emulator from ProgramFolder path is faster - Unnecessary exponential retry logic (current one)
# Pull Request Template
## Description
This PR adds optional attributes in the `VectorIndexDefinition` class to
support partitioned DiskANN. A typical index definition would be
something like the below:
```
{
"indexingPolicy": {
"automatic": true,
"indexingMode": "Consistent",
"includedPaths": [
{
"path": "/*",
"indexes": []
}
],
"excludedPaths": [],
"compositeIndexes": [],
"spatialIndexes": [],
"vectorIndexes": [
{
"path": "/vector1",
"type": "flat"
},
{
"path": "/vector2",
"type": "quantizedFlat",
"quantizationByteSize": 3,
"vectorIndexShardKey": [
"/Country"
]
},
{
"path": "/vector3",
"type": "diskANN",
"quantizationByteSize": 2,
"indexingSearchListSize": 100,
"vectorIndexShardKey": [
"/ZipCode"
]
}
]
},
"vectorEmbeddingPolicy": {
"vectorEmbeddings": [
{
"path": "/vector1",
"dataType": "int8",
"dimensions": 1200,
"distanceFunction": "dotproduct"
},
{
"path": "/vector2",
"dataType": "uint8",
"dimensions": 3,
"distanceFunction": "cosine"
},
{
"path": "/vector3",
"dataType": "float32",
"dimensions": 400,
"distanceFunction": "euclidean"
}
]
},
"id": "test_binary_vector_container_6",
"partitionKey": {
"paths": [
"/pk"
],
"kind": "Hash"
}
}
```
## Type of change
Please delete options that are not relevant.
- [x] New feature (non-breaking change which adds functionality)
## Closing issues
To automatically close an issue: closes #4628
---------
Co-authored-by: Kiran Kumar Kolli <[email protected]>
Azure.Core: Fixed upgrading azure core dependency to 1.44.1 Changes - ResourceType: Conflict between Azure.Core and Microsoft.Azure.Cosmos.Documents - Microsoft.Bcl.AsyncInterfaces: Azure.Core needs at-least 6.0.0 (as Azure core upgraded part of minor version, we are good as well) - Microsoft.Azure.Cosmos.Encryption.Custom: Direct dependency on Azure.Core removed (now its transitive) - Microsoft.Azure.Cosmos.Encryption.Custom.Performance.Tests: `NU1903` added Newtonsoft.Json dependency - Performance project: `NU1903` Newtonsoft dependency upgraded
…onKeyStreamAsync API to GA (#4814) DeleteAllItemsByPartitionKeyStreamAsync: Adds DeleteAllItemsByPartitionKeyStreamAsync API to GA Changes - DeleteAllItemsByPartitionKeyStreamAsync: Marked virtual with default NotSupportedException (Dependent libraries will continue to work) - Encryption package related changes will be in a follow-up PR after the package is published
FabianMeiswinkel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move to static values for custom dimensions
FabianMeiswinkel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Thanks!
FabianMeiswinkel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM Thanks
Description
Introduced new options in
ClientTelemetryOptionsclass to configure operation-level and network-level metrics at both the client and request levels:OperationMetricsOptions
trueat either the client or request level.NetworkMetricsOptions
trueat either the client or request level.Type of change