Skip to content

Update identity guardrail language to add clarity #447

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
nhecht opened this issue Apr 22, 2025 · 1 comment
Open

Update identity guardrail language to add clarity #447

nhecht opened this issue Apr 22, 2025 · 1 comment

Comments

@nhecht
Copy link
Contributor

nhecht commented Apr 22, 2025

Issue in ./help/identity-service/guardrails.md

| Number of links to an identity for a single batch ingestion | 50 | A single batch could contain anomalous identities that cause unwanted graph merges. To prevent this, Identity Service will not ingest identities that are already linked to 50 or more identities. |

The following line creates some ambiguity on how this is handled.

I have followed up with Engineering to confirm the behavior.

Suggest the Notes be updated to say the following.

"If a single identity in a batch is linked to 50 or more other identities in the batch, the identity is considered invalid, and will be filtered out of the batch linkages."

This is more clear then the current language.
"A single batch could contain anomalous identities that cause unwanted graph merges. To prevent this, Identity Service will not ingest identities that are already linked to 50 or more identities."

One could interpret the above to mean if the identity already had 50 linkages in the prior ingested graph. But the batch identity ingestion process makes no reference to the prior ingested graph status.

@vfalon
Copy link
Collaborator

vfalon commented Apr 24, 2025

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-229027.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants