Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Append UDF with RDD or RF. #381

Merged
merged 1 commit into from Nov 19, 2019

Conversation

@ruebot
Copy link
Member

ruebot commented Nov 19, 2019

GitHub issue(s): #223

What does this Pull Request do?

Append UDF with RDD or RF.

How should this be tested?

  • TravisCI (updated unit tests) should take care of it.

Additional Notes:

There might be some documentation updates to do.

- Addresses #223
@ruebot ruebot requested a review from lintool Nov 19, 2019
@ruebot

This comment has been minimized.

Copy link
Member Author

ruebot commented Nov 19, 2019

@SinghGursimran once we get this merged, this should be a solid implementation pattern for you as we sort out the rest of #223.

@lintool all the names make me cringe a bit, but it makes things a lot clearer now.

@codecov

This comment has been minimized.

Copy link

codecov bot commented Nov 19, 2019

Codecov Report

Merging #381 into master will not change coverage.
The diff coverage is 98.11%.

@@           Coverage Diff           @@
##           master     #381   +/-   ##
=======================================
  Coverage   76.23%   76.23%           
=======================================
  Files          40       40           
  Lines        1422     1422           
  Branches      268      268           
=======================================
  Hits         1084     1084           
  Misses        221      221           
  Partials      117      117
@ruebot

This comment has been minimized.

Copy link
Member Author

ruebot commented Nov 19, 2019

Oh. Lots of documentation updates to do. I've started a local branch.

ruebot added a commit to archivesunleashed/aut-docs-new that referenced this pull request Nov 19, 2019
@ruebot

This comment has been minimized.

Copy link
Member Author

ruebot commented Nov 19, 2019

Copy link
Member

ianmilligan1 left a comment

LGTM - the names are messy, as you note, but it'll keep things straight.

@ianmilligan1 ianmilligan1 merged commit b98ba4b into master Nov 19, 2019
3 checks passed
3 checks passed
codecov/patch 98.11% of diff hit (target 76.23%)
Details
codecov/project 76.23% (+0%) compared to a081d7b
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ianmilligan1 ianmilligan1 deleted the rdd-df branch Nov 19, 2019
ianmilligan1 added a commit to archivesunleashed/aut-docs-new that referenced this pull request Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.