Skip to content

Conversation

kurisumakise2011
Copy link
Contributor

@kurisumakise2011 kurisumakise2011 commented Jun 24, 2020

Increase length of debug string to 36, due to UUID length. According to https://tools.ietf.org/html/rfc4122#section-3 a UUID should be represented as 36 characters (32 hex digits + 4 dashes) in string format. This makes it more convenient for debugging purposes.

Fixes #300

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 24, 2020
Increase length of debug string to 36, due to UUID length.
According to https://tools.ietf.org/html/rfc4122#section-3
It provides the formal definition of UUID string representations.
It's 36 characters (32 hex digits + 4 dashes)
It should be more convenient to debugging statements
@olavloite olavloite changed the title feature: Increase a length of debug string feat: increase the max length of debug string Jun 24, 2020
@olavloite olavloite requested review from olavloite and skuruppu June 24, 2020 20:06
Copy link
Collaborator

@olavloite olavloite left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. This looks good to me.

@olavloite olavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 24, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 24, 2020
@skuruppu skuruppu merged commit 6a900aa into googleapis:master Jun 25, 2020
ansh0l pushed a commit to ansh0l/java-spanner that referenced this pull request Nov 10, 2022
This is an auto-generated regeneration of the .pb.go files by
cloud.google.com/go/internal/gapicgen. Once this PR is submitted, genmgr will
update the corresponding CL at gocloud to depend on the newer version of
go-genproto, and assign reviewers. Whilst this or any regen PR is open in
go-genproto, gapicgen will not create any more regeneration PRs or CLs. If all
regen PRs are closed, gapicgen will create a new set of regeneration PRs and
CLs once per night.

If you have been assigned to review this CL, please:

- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship. That will prompt
  genmgr to assign reviewers to the gocloud CL.

Corresponding gocloud CL: https://code-review.googlesource.com/c/gocloud/+/51650
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increasing the number of MAX_DEBUG_STRING_LENGTH from 32 to 36 due to cut uuid representation

5 participants