-
-
Notifications
You must be signed in to change notification settings - Fork 191
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
Schema-qualified table names #2466
Conversation
…nto zachmu/schemas
… case of indexed table access, some other table node types
…ndled by callers of RowCount. This means that table functions which don't have stats get treated as the default row count size by the coster
…nto zachmu/schemas
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, nothing really looks suspicious, other than whatever was going on with FK resolution before.
It turns out that Dolt required the table to be re-resolved because of delayed foreign key resolution, which means that an update statement has the side effect of updating the serialized schema of a table :\ I fixed it in the Dolt layer. |
This PR also fixes a couple unrelated issues: