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.

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

Rethink definition of variables in the result of a join #3425

Closed
krlmlr opened this issue Mar 14, 2018 · 1 comment · Fixed by #3430
Closed

Rethink definition of variables in the result of a join #3425

krlmlr opened this issue Mar 14, 2018 · 1 comment · Fixed by #3430
Labels
feature a feature request or enhancement verbs 🏃‍♀️

Comments

@krlmlr
Copy link
Member

krlmlr commented Mar 14, 2018

Simplify name generation and resolution of clashes:

  1. Use variables, remove rhs variables for equi-joins (but not for generic joins)
  2. Add suffixes to remaining variables of the same name in lhs and rhs
  3. Combine variables: join variables, remaining lhs variables, remaining rhs variables
  4. Apply tibble::set_tidy_names().

For dplyr, the result should be two lists that can be unquote-spliced into select(). The C++ side then only needs to know the number of join variables and return the result, For dbplyr, something similar to the current join_vars() implementation would be useful. Need to think about a data structure that supports both, and export the function that computes the join variables.

Useful for/part of #2240.

@lock
Copy link

lock bot commented Sep 12, 2018

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/

@lock lock bot locked and limited conversation to collaborators Sep 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature a feature request or enhancement verbs 🏃‍♀️
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant