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

[skip-travis] Add spark-submit option to README; resolves #444. #446

Merged
merged 2 commits into from Apr 15, 2020

Conversation

@ruebot
Copy link
Member

ruebot commented Apr 15, 2020

GitHub issue(s): #444

What does this Pull Request do?

Add spark-submit option to README.

How should this be tested?

Let me know if this works. Tried to get it fairly simply, and just link out to the more robust documentation. Link.

@ruebot ruebot requested review from ianmilligan1 and SamFritz Apr 15, 2020
README.md Outdated
The Toolkit offers a variety of extraction jobs with
[`spark-submit`](https://spark.apache.org/docs/latest/submitting-applications.html)
. These extraction jobs have a few configuration options, and analysis can use
RDD or DataFrame is most cases.

This comment has been minimized.

Copy link
@ianmilligan1

ianmilligan1 Apr 15, 2020

Member

minor typo
is -> in

@ianmilligan1

This comment has been minimized.

Copy link
Member

ianmilligan1 commented Apr 15, 2020

^^^ Oops, hit "approved" but did catch one minor typo @ruebot (apologies on the butterfinger). But this is great, and it really occurs to me that we should be promoting spark-submit more as this is so straightforward.

@codecov

This comment has been minimized.

Copy link

codecov bot commented Apr 15, 2020

Codecov Report

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

@@           Coverage Diff           @@
##           master     #446   +/-   ##
=======================================
  Coverage   75.55%   75.55%           
=======================================
  Files          40       40           
  Lines        1395     1395           
  Branches      265      265           
=======================================
  Hits         1054     1054           
  Misses        218      218           
  Partials      123      123           
@ianmilligan1 ianmilligan1 merged commit d5a0433 into master Apr 15, 2020
3 checks passed
3 checks passed
codecov/patch Coverage not affected when comparing 35291bf...52a80c4
Details
codecov/project 75.55% (+0.00%) compared to 35291bf
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ianmilligan1 ianmilligan1 deleted the issue-444 branch Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.