Skip to content

Fix BITCOUNT() for long binary strings #3647

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.

Sign up for GitHub

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

Jump to bottom
Merged
katzyn merged 2 commits into from
Oct 10, 2022
Merged

Fix BITCOUNT() for long binary strings #3647

katzyn merged 2 commits into from
Oct 10, 2022

Conversation

@katzyn
Copy link
Contributor

@katzyn katzyn commented Oct 10, 2022

Closes #3645.

@sonatype-lift
Copy link
Contributor

sonatype-lift bot commented Oct 10, 2022

⚠️ 221 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@katzyn
Copy link
Contributor Author

katzyn commented Oct 10, 2022

@grandinj
Is there a way to silence that “God Classes” warning?

@katzyn katzyn merged commit 7c6fab9 into h2database:master Oct 10, 2022
@katzyn katzyn deleted the bitcount branch October 10, 2022 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

The BITCOUNT function incorrectly counts BINARY/VARBINARY values of >=16 bytes.

1 participant

@katzyn