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

Add criteria for max separate sync calls to apply when flushing #579

Merged
merged 3 commits into from
Jul 29, 2024

Conversation

gruuya
Copy link
Contributor

@gruuya gruuya commented Jul 26, 2024

This is to avoid the stack overflow during plan tree walking in optimizations/physical plan conversion etc.

@gruuya gruuya requested a review from mildbyte July 26, 2024 14:27
gruuya added 2 commits July 29, 2024 09:01
The test overflows the stack in debug mode, but passes in release mode.
@gruuya gruuya merged commit 8295ba6 into main Jul 29, 2024
1 check passed
@gruuya gruuya deleted the max-sync-call-criterium branch July 29, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants