Azure / Azure/azure-sdk-for-rust

Cosmos Driver: Validate and Fix Go/ Rust EPK Test Gaps

Closed
#4,054 1 comment 0 reactions 1 assignee Claimed by @simorenoh View on GitHub
Cosmos
Dominant language
Rust
Stars
884
Forks
365
Avg merge
2d 19h
Merged PRs (30d)
112

Description

- The goal is to test that not only is the murmurhash implementation correct, but our encoding is correct as well. We should use the Rust SDKs full PartitionKey -> EPK pipeline, and not just reimplement Go's encoding here. Go's encoding logic SHOULD be the same as Rust's, and Python's, and .NET's, and Java's. That's the intent of this test.

```
let pk = PartitionKey::from(...);
let epk = EffectivePartitionKey::compute(pk);
assert_eq!(baseline_value, epk.as_str())
```
- Add Integration tests for pk_range_cache. It would be good if the cache is filled proactively when creating container client It would also help for removing many of the dead code snippets for this iteration.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.