Skip to content

feat: arrayofnumbers compact sketch - #89

Merged
freakyzoidberg merged 3 commits into
apache:mainfrom
proost:feat-arrayofnumbers-compact-sketch
Jan 7, 2026
Merged

freakyzoidberg merged 3 commits into
apache:mainfrom
proost:feat-arrayofnumbers-compact-sketch

Conversation

@proost

@proost proost commented Jan 2, 2026

Copy link
Copy Markdown
Member

issue: #79

adding ArrayOfNumbers compact sketch & compatability tests.

@freakyzoidberg freakyzoidberg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM - if you can fix the family definition location.

Thanks

Comment thread tuple/arrayofnumbers_compact_sketch.go Resolved
Comment thread tuple/arrayofnumbers_compact_sketch.go Resolved
@proost
proost requested a review from freakyzoidberg January 6, 2026 16:05
@proost

proost commented Jan 6, 2026

Copy link
Copy Markdown
Member Author

@freakyzoidberg
I will add theta family enum in the another PR.

@freakyzoidberg
freakyzoidberg merged commit fecb45e into apache:main Jan 7, 2026
1 check passed
@proost
proost deleted the feat-arrayofnumbers-compact-sketch branch January 7, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants