Skip to content
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.

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

Fix spurious function compilation error #21686

Conversation

findepi
Copy link
Member

@findepi findepi commented Apr 24, 2024

Before the change, the function compiler sometimes failed with "break target does not exist" exception. This was reproducible only in distributed execution.

Fixes #21682

@cla-bot cla-bot bot added the cla-signed label Apr 24, 2024
Before the change, the function compiler sometimes failed with "break
target does not exist" exception. This was reproducible only in
distributed execution.
@findepi findepi force-pushed the findepi/fix-spurious-function-compilation-error-18946a branch from 5bed93f to 43b64be Compare April 24, 2024 12:05
@findepi findepi marked this pull request as draft April 24, 2024 14:29
private final Map<IrLabel, LabelNode> continueLabels = new IdentityHashMap<>();
private final Map<IrLabel, LabelNode> breakLabels = new IdentityHashMap<>();
private final Map<IrLabel, LabelNode> continueLabels = new HashMap<>();
private final Map<IrLabel, LabelNode> breakLabels = new HashMap<>();
Copy link
Member Author

Choose a reason for hiding this comment

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

this won't work when function body has same label multiple times.
see TestSqlFunctions.testReuseLabels

@findepi
Copy link
Member Author

findepi commented Apr 24, 2024

superseded by #21691

@findepi findepi closed this Apr 24, 2024
@findepi findepi deleted the findepi/fix-spurious-function-compilation-error-18946a branch April 24, 2024 15:59
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.

Function with LOOP fails with "Compiler failed [...] break target does not exist" when run on worker
2 participants