-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
colexec: re-enable short-circuiting of hash join #49631
Comments
Hm, I wonder whether this timeout is somehow due to the limit being too low. The most suspicious goroutine in the stack trace is
|
Which limit do you mean? |
I meant the time limit of 24 hours. |
Might be, but we've had successful runs of 12-18h so it would be a bit surprising. Although given that variance maybe a 24h run is to be expected? |
Let's focus on fixing |
DrainMeta/Next issue has been fixed, I think the next step is fixing Closers and then enabling the short-circuiting in the hash joiner. I'll assign the issue to @asubiotto since you're working on that. |
(sql/logictest).TestSqlLiteLogic/local/slt_good_1.test#09 failed on master@d8a0f6c2ad31df7f5dc34b5e1bd74bc2f056c2cd:
More
See this test on roachdash
powered by pkg/cmd/internal/issues
The text was updated successfully, but these errors were encountered: