Skip to content

Add constants for UUID lengths#112353

Merged
elasticsearchmachine merged 2 commits intoelastic:mainfrom
DaveCTurner:2024/08/29/uuid-length
Aug 29, 2024
Merged

Add constants for UUID lengths#112353
elasticsearchmachine merged 2 commits intoelastic:mainfrom
DaveCTurner:2024/08/29/uuid-length

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Our UUID strings have fixed lengths (depending on the type of UUID).
Sometimes we might want code to rely on knowing these lengths rather
than doing some other string manipulations to look for a boundary. This
commit exposes constants for these things.

Our UUID strings have fixed lengths (depending on the type of UUID).
Sometimes we might want code to rely on knowing these lengths rather
than doing some other string manipulations to look for a boundary. This
commit exposes constants for these things.
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Aug 29, 2024
Copy link
Copy Markdown
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@DaveCTurner DaveCTurner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 29, 2024
@elasticsearchmachine elasticsearchmachine merged commit 1be4f65 into elastic:main Aug 29, 2024
@DaveCTurner DaveCTurner deleted the 2024/08/29/uuid-length branch August 29, 2024 14:20
dakrone pushed a commit to dakrone/elasticsearch that referenced this pull request Aug 30, 2024
Our UUID strings have fixed lengths (depending on the type of UUID).
Sometimes we might want code to rely on knowing these lengths rather
than doing some other string manipulations to look for a boundary. This
commit exposes constants for these things.
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this pull request Sep 4, 2024
Our UUID strings have fixed lengths (depending on the type of UUID).
Sometimes we might want code to rely on knowing these lengths rather
than doing some other string manipulations to look for a boundary. This
commit exposes constants for these things.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants