Uh oh!
There was an error while loading. Please reload this page.
Use uuid, it speeds up the call - #4786
Conversation
I have read the CLA Document and I hereby sign the CLA 1 out of 2 committers have signed the CLA. |
JorjMcKie
commented
Nov 12, 2025
I've reviewed your proposal, but I don't believe it addresses a significant performance concern. Our current implementation achieves durations in the low single-digit microsecond range — even with name lists of around 20 entries, which is itself uncommon. When using a set() for containment checks, performance nearly doubles and matches or exceeds that of a UUID-based approach when the list size is 10 or fewer — the most typical case. Additionally, the current naming strategy produces short and readable identifiers in contrast to UUIDs like 'b58cb87d-50a0-4ae2-be88-77ea77afdb0d'. Given these factors, we’ll stick with the existing solution and ensure we will use the |
yuval-peleg-levy-ai21
commented
Nov 12, 2025
Hey, I did this after seeing these calls are a bottlenck using a profiler. Thanks! |
JorjMcKie
commented
Nov 12, 2025
Ah, now I understand your motivation better! |
yuval-peleg-levy-ai21
commented
Nov 13, 2025
Thanks. Sorry for not being more detailed in my PR. |
Use uuid, it speeds up the call