You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Makes sense and it should not be that hard to do.
Probably we should have something like internal normalize_on function that rewrites any on argument originally passed to be a Vector{Pair}, so that the later code can safely assume only one form (and this operation should be cheap relative to the other things join does).
Then in normalize_on we could make all the checks that the passed argument is valid.
Would you want to make a PR (if you do not have time I will have a look at this).
The only thing we have to be careful about when implementing it is that on master we allow joining more than two data frames in a single call (in which case we do not allow Pairs).
Currently we can do:
but it's not possible to do:
This would be very convenient for cases where one has a large number of columns to match on, with only one or two that don't match and require a
Pair
.The text was updated successfully, but these errors were encountered: