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

[R-Forge #5712] Error message for "allow.cartesian=TRUE" #508

Closed
arunsrinivasan opened this issue Jun 8, 2014 · 2 comments
Closed

[R-Forge #5712] Error message for "allow.cartesian=TRUE" #508

arunsrinivasan opened this issue Jun 8, 2014 · 2 comments
Assignees
Milestone

Comments

@arunsrinivasan
Copy link
Member

Submitted by: Arun ; Assigned to: Nobody; R-Forge link

The error message suggesting to use allow.cartesian=TRUE currently is:

# Error in vecseq(f__, len__, if (allow.cartesian) NULL else as.integer(max(nrow(x),  : 
#   Join results in 320342 rows; more than 160171 = max(nrow(x),nrow(i)). Check for duplicate key 
# values in i, each of which join to the same group in x over and over again. If that's ok, try including `j` 
# and dropping `by` (by-without-by) so that j runs for each group to avoid the large allocation. If you 
# are sure you wish to proceed, rerun with allow.cartesian=TRUE. Otherwise, please search for this 
# error message in the FAQ, Wiki, Stack Overflow and datatable-help for advice.

However, this also happens when x has duplicate values but not i. Check eddi's post here for an example.

So, it's a bit misleading to check for i having duplicate keys.. Not sure how to change it yet. Filing so that we don't miss it.

@arunsrinivasan arunsrinivasan added this to the v1.9.4 milestone Jun 19, 2014
@arunsrinivasan arunsrinivasan modified the milestones: 2.0.2, v2.0 Sep 6, 2014
@arunsrinivasan
Copy link
Member Author

This needs to be a FAQ as well. Will do before to close.

@mattdowle
Copy link
Member

#742 was great fix, Arun! No need for FAQ.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants