Skip to content

Fix ClassNotFoundException in state compiler#10346

Merged
dain merged 2 commits intotrinodb:masterfrom
dain:fix-cnf-in-state-compiler
Dec 19, 2021
Merged

Fix ClassNotFoundException in state compiler#10346
dain merged 2 commits intotrinodb:masterfrom
dain:fix-cnf-in-state-compiler

Conversation

@dain
Copy link
Copy Markdown
Member

@dain dain commented Dec 18, 2021

Fixes #10341

@dain dain requested review from electrum and martint December 18, 2021 22:51
@cla-bot cla-bot bot added the cla-signed label Dec 18, 2021
@dain dain merged commit 744ee5e into trinodb:master Dec 19, 2021
@dain dain deleted the fix-cnf-in-state-compiler branch December 19, 2021 23:47
@github-actions github-actions bot added this to the 367 milestone Dec 19, 2021
@findepi
Copy link
Copy Markdown
Member

findepi commented Dec 20, 2021

We should have a product test covering @realknorke 's case (#10341) to prevent regressions. I don't think we're able to effectively exercise classloader-related issues in a unit test manner.

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.

DynamicClassLoader regression bug breaks simple (external) plugins

3 participants