Skip to content

GH-1158: Fix inclusive bounds checking for DictionaryDecoder - #1159

Closed
jakobhok-neo4j wants to merge 1 commit into
apache:mainfrom
jakobhok-neo4j:gh-1158
Closed

jakobhok-neo4j wants to merge 1 commit into
apache:mainfrom
jakobhok-neo4j:gh-1158

Conversation

@jakobhok-neo4j

@jakobhok-neo4j jakobhok-neo4j commented May 19, 2026 •

Copy link
Copy Markdown

Closes #1158.

@jakobhok-neo4j
jakobhok-neo4j marked this pull request as ready for review May 19, 2026 06:34
@github-actions

This comment has been minimized.

@lidavidm lidavidm added the bug-fix PRs that fix a big. label Aug 25, 2026
@github-actions github-actions Bot added this to the 20.0.0 milestone Aug 25, 2026

@lidavidm lidavidm 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.

Seems reasonable. Can you rebase to pick up CI fixes?

@lidavidm

Copy link
Copy Markdown
Member

Thanks, this still has conflicts though

@jakobhok-neo4j

Copy link
Copy Markdown
Author

Ah, rebased on wrong branch. mb, but it looks like it is already fixed. I think we can close it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix PRs that fix a big.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Java] DictionaryEncoder doesn't crash when decoding index outside of Dictionary

2 participants