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.

Sign up for GitHub

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

Jump to bottom

sql: instrument lateral joins #42470

Closed
awoods187 opened this issue Nov 14, 2019 · 0 comments · Fixed by #42789
Closed

sql: instrument lateral joins #42470

awoods187 opened this issue Nov 14, 2019 · 0 comments · Fixed by #42789
Assignees
@RaduBerinde
Labels
A-sql-optimizer SQL logical planning and optimizations. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Milestone
19.2.x

Comments

@awoods187
Copy link
Contributor

Now that we have added support for lateral joins (#24560) we should instrument them in telemetry
@awoods187 awoods187 added A-sql-optimizer SQL logical planning and optimizations. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) labels Nov 14, 2019
@RaduBerinde RaduBerinde added this to the 19.2.x milestone Nov 14, 2019
@RaduBerinde RaduBerinde mentioned this issue Nov 26, 2019
Merged
craig bot pushed a commit that referenced this issue Nov 26, 2019
@RaduBerinde
Merge #42789
916ab35 
42789: opt: add telemetry for LATERAL and WITH RECURSIVE r=RaduBerinde a=RaduBerinde

Fixes #42470.
Fixes #42469.

Release note: None

Co-authored-by: Radu Berinde <radu@cockroachlabs.com>
@craig craig bot closed this as completed in c3664fd Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RaduBerinde RaduBerinde

Labels
A-sql-optimizer SQL logical planning and optimizations. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Projects
None yet
Milestone
19.2.x
Development

Successfully merging a pull request may close this issue.

opt: add telemetry for LATERAL and WITH RECURSIVE RaduBerinde/cockroach
2 participants
@RaduBerinde @awoods187