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

update on join by join field no longer errors, closes #3559 #2099 #3807

Merged
merged 4 commits into from
Sep 4, 2019

Conversation

jangorecki
Copy link
Member

@jangorecki jangorecki commented Sep 1, 2019

closes #3559
closes #2099 (no tests added for that because reported code is not that minimal and boils down to factor issue which is tested)

@codecov
Copy link

codecov bot commented Sep 1, 2019

Codecov Report

Merging #3807 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3807   +/-   ##
=======================================
  Coverage   99.41%   99.41%           
=======================================
  Files          71       71           
  Lines       13231    13231           
=======================================
  Hits        13154    13154           
  Misses         77       77

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1e23356...be5e411. Read the comment docs.

@jangorecki jangorecki changed the title update on join by join field no longer errors, closes #3559 update on join by join field no longer errors, closes #3559 #2099 Sep 1, 2019
@mattdowle mattdowle merged commit e795495 into master Sep 4, 2019
@mattdowle mattdowle deleted the join-on-factor-sd branch September 4, 2019 21:03
@MichaelChirico
Copy link
Member

nice!

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

Successfully merging this pull request may close these issues.

Error in update join when joining on factor .SD locked error using foverlaps after ordering by variable
3 participants