Conversation
Two design documents for community discussion, not intended to merge: restructure-draft-v2.md defines the target file structure and namespace hierarchy for 6.0.0 -- a single include root, one directory and nested namespace per sketch family, and an internal/ subdirectory for everything outside the supported API. It records 18 ratified decisions and 3 open items. restructure-plan-detail.md holds the sequencing: the PR order from the C++17 flag flip through the move, the namespace rewrite and cleanup, plus a per-area breakdown and the techniques for keeping each PR reviewable. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Satisfies the RAT license audit. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
PointKernel
reviewed
Sep 23, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two design documents for the 6.0.0 restructure, posted here so they can be commented on line by line. This PR is not intended to merge — it is a place to mark up the text. The decision thread will be on dev@datasketches.apache.org
restructure-draft-v2.md (pretty copy, the markup review copy is below) — the target file structure and namespace hierarchy: a single include root, one directory and nested namespace per sketch family, and an
internal/subdirectory for everything outside the supported API. Every header in the current tree is accounted for: kept, moved, renamed, split, or deleted. Includes the precedent in other libraries, 18 ratified decisions and 3 open items.restructure-plan-detail.md (pretty copy, the markup review copy is below) — the sequencing: the PR order from the C++17 flag flip through the move, the namespace rewrite and the cleanup, plus a per-area breakdown and the techniques for keeping each PR reviewable.
Comments inline on the files are very welcome, particularly on the ratified decisions and the open items.
🤖 Generated with Claude Code