Skip to content
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

Draft 03, Phase 4 PR #85

Merged
merged 3 commits into from
Mar 16, 2022
Merged

Draft 03, Phase 4 PR #85

merged 3 commits into from
Mar 16, 2022

Conversation

kyzer-davis
Copy link
Contributor

@kyzer-davis kyzer-davis commented Mar 9, 2022

3-9-2022 changelog:


Reviewers:

  • Please discuss changes in this PR within the threads above to keep discussions grouped nicely together.

@bradleypeabody
Copy link
Contributor

Thanks @kyzer-davis generally this is shaping up but I worry about this UUID 7 vs 7E, etc. - I think we need to coordinate further on this point. I'll be in touch on this. Just from a quick review the rest of it overall makes sense and probably any other issues I have could be addressed with minor edits which I can follow up with later after I've reviewed in more detail (this thing is still changing so quickly I haven't had a chance to read it from start to end without it changing as I'm reading it, lol - but hey, it's definitely shaping up so no worries.)

Let's chat about this 7 vs 7E thing and then decide if this should be merged as is or what.

@kyzer-davis
Copy link
Contributor Author

@bradleypeabody,

All good.
I have been burning the candle at both ends and iterating on the text daily based on the discussion threads in order to get this text ready for March 31st IETF resubmission.

@kyzer-davis
Copy link
Contributor Author

Updated Fixed Length Counters for N-1 Technique over zero start as per #60

@kyzer-davis
Copy link
Contributor Author

I have removed E Variant, Alt Encoding and UUID Long sections from the document.
This XML has been retained and will be used for the start of a new draft on these 3 topics.

@kyzer-davis kyzer-davis merged commit 606736c into master Mar 16, 2022
@kyzer-davis kyzer-davis deleted the draft-03-phase-4 branch March 16, 2022 21:47
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

Successfully merging this pull request may close these issues.

Draft 03: Recommend Database Side UUID Generation
2 participants