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
Cleanup of join code #2514
Cleanup of join code #2514
Conversation
- always expand factor to character vector - join compares and hashes `CHARSXP` instead of matching via `r_match()`, no special handling needed! - now relying on declared encoding, converting to UTF-8 if necessary
@hadley: This gives a ~3-4x speedup for joining strings, if they don't need to be reencoded. |
Nice! |
) | ||
} | ||
|
||
if (R_version != R.version.string) { |
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.
Do you remember why you added this check here?
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.
We're checking the R version at compile time, currently only for a workaround for a problem in R 3.3.0 .
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/ |
in preparation for #2033. Considerably simplifies string-string and string-factor joins.
Fixes #1858.