apunkt/omnigraph
1
0
Fork 0
mirror of https://github.com/ModernRelay/omnigraph.git synced 2026-06-12 01:45:14 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

docs: clarify unit-separator comment in composite unique check

Browse source
This commit is contained in:
Ragnor Comerford 2026-05-31 02:11:44 +00:00
parent 779a6a039e
commit 11c03b90a1
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
crates/omnigraph/src/loader/mod.rs
View file

@ -1463,8 +1463,9 @@ pub(crate) fn enforce_unique_constraints_intra_batch(
if any_null {
continue;
}
// Unit separator (U+001F) keeps composite parts unambiguous; it
// cannot appear in the scalar renderings of supported types.
// Join on the unit separator (U+001F) — a control char highly
// unlikely to occur in real data, keeping composite keys
// effectively unambiguous (cf. `exec/merge.rs`, which uses `|`).
let value = parts.join("\u{1f}");
if let Some(prev_row) = seen.insert(value.clone(), row) {
return Err(OmniError::manifest(format!(