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.

Sign up for GitHub

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

Jump to bottom

[R-Forge #2468] merge() should warn if either table is empty #597

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

[R-Forge #2468] merge() should warn if either table is empty #597

arunsrinivasan opened this issue Jun 8, 2014 · 0 comments
Labels
feature request

Comments

@arunsrinivasan
Copy link
Member

Submitted by: Matt Dowle; Assigned to: Nobody; R-Forge link

or, at least before any type error occurs. if sqlQuery downloads 0 rows, it still has all the columns but the types haven't been able to detected by sqlQuery inspecting the data (so all left as type logical(0)). That can generate type error by a keyed join that normally works that expects the key column to be character (as it is normally when some rows have been downloaded) rather than logical, for example.
jangorecki added a commit that referenced this issue Feb 25, 2016
@jangorecki
Merge pull request #1546 from jangorecki/merge-empty-dt-warn
7a9b800 
merge.data.table will raise warning for 0 columns data.tables. Closes #597
arunsrinivasan added a commit that referenced this issue Feb 25, 2016
@arunsrinivasan
Merge branch 'master' of https://github.com/Rdatatable/data.table
52d26c3 
* 'master' of https://github.com/Rdatatable/data.table:
  merge.data.table will raise warning for 0 columns data.tables. Closes #597
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@arunsrinivasan