-
Notifications
You must be signed in to change notification settings - Fork 5k
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
[MLv2] joinable-columns
on a Join sets source and alias
#39529
Conversation
`(joinable-columns query stage a-join)` previously returned `:lib/source :source/table-defaults` and set `:metabase.lib.join/join-alias` but not `:source-alias`. This corrects both issues so that `joinable-columns` on a join returns columns with the same shape as from `visible-columns`. Also refactors a series of `mapv` transforms into a transducer pipeline. Fixes #39514.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @bshepherdson and the rest of your teammates on |
|
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. 👍 for using transducers instead of multiple sequence transformations.
@bshepherdson Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone? |
`(joinable-columns query stage a-join)` previously returned `:lib/source :source/table-defaults` and set `:metabase.lib.join/join-alias` but not `:source-alias`. This corrects both issues so that `joinable-columns` on a join returns columns with the same shape as from `visible-columns`. Also refactors a series of `mapv` transforms into a transducer pipeline. Fixes #39514.
…39543) `(joinable-columns query stage a-join)` previously returned `:lib/source :source/table-defaults` and set `:metabase.lib.join/join-alias` but not `:source-alias`. This corrects both issues so that `joinable-columns` on a join returns columns with the same shape as from `visible-columns`. Also refactors a series of `mapv` transforms into a transducer pipeline. Fixes #39514. Co-authored-by: Braden Shepherdson <braden@metabase.com>
(joinable-columns query stage a-join)
previously returned:lib/source :source/table-defaults
and set:metabase.lib.join/join-alias
but not:source-alias
.This corrects both issues so that
joinable-columns
on a join returnscolumns with the same shape as from
visible-columns
.Also refactors a series of
mapv
transforms into a transducer pipeline.Fixes #39514.