Skip to content

Cleanup in TranslationMap#28593

Open
wendigo wants to merge 3 commits intotrinodb:masterfrom
wendigo:user/serafin/translation-map-cleanup
Open

Cleanup in TranslationMap#28593
wendigo wants to merge 3 commits intotrinodb:masterfrom
wendigo:user/serafin/translation-map-cleanup

Conversation

@wendigo
Copy link
Contributor

@wendigo wendigo commented Mar 9, 2026

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Mar 9, 2026
@wendigo wendigo requested a review from martint March 9, 2026 23:04
}

return call;
case null, default -> throw new IllegalArgumentException("Unexpected type: " + valueType);
Copy link
Member

Choose a reason for hiding this comment

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

valueType should never be null

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

Development

Successfully merging this pull request may close these issues.

2 participants