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

Add Apache Spark #895

Closed
wants to merge 1 commit into from
Closed

Add Apache Spark #895

wants to merge 1 commit into from

Conversation

zero323
Copy link

@zero323 zero323 commented Mar 2, 2017

https://github.com/awesome-spark/awesome-spark

Apache Spark is a popular data processing framework and of the most active open source projects today.

By submitting this pull request I confirm I've read and complied with the below requirements.

Failure to properly do so will just result in the pull request being closed and everyone's time wasted. Please read it twice. Most people miss many things.

  • I have read and understood the contribution guidelines and the instructions for creating a list.
  • This pull request has a descriptive title. For example, Add Name of List, not Update readme.md or Add awesome list.
  • The entry in the Awesome list should include a short description about the list project/theme (not the list itself), and should be added at the bottom of the appropriate category.
  • The list I added
    • has been around for at least 30 days,
    • is the result of hard work and is the best I could possibly produce,
    • is a non-generated Markdown file in a GitHub repo,
    • is not a duplicate,
    • includes a succinct description of the project/theme at the top (example),
    • includes a project logo/illustration (whenever possible) at the top right of the readme (example). (The logo should link to the project website or any relevant website. The logo should be high-DPI, so set it to maximum half the width of the original image.),
    • only has awesome items with descriptions (Awesome lists are curations of the best, not of everything),
    • contains the awesome badge on the right side of the list heading,
    • has a Table of Contents section named Contents as the first section,
    • has an appropriate license (Meaning something like CC0, NOT a code licence like MIT, BSD, Apache, etc) (If you use a license badge, it should be SVG, not PNG),
    • has contribution guidelines,
    • doesn't include a Travis badge,
    • has been checked for inconsistent formatting, spelling, and grammar:
      • Each link description starts with an uppercase character and ends with a period.
      • Drop all the A / An prefixes in the descriptions.
      • Consistent naming. For example, Node.js, not NodeJS or node.js.

@zero323 zero323 changed the title Add Awesome (Apache) Spark Add (Apache) Spark Mar 2, 2017
@eliasah
Copy link

eliasah commented Mar 3, 2017

LGTM

readme.md Outdated
@@ -253,6 +253,7 @@ Check out my [blog](https://blog.sindresorhus.com) and follow me on [Twitter](ht
- [Hadoop](https://github.com/youngwookim/awesome-hadoop) - Framework for distributed storage and processing of very large data sets.
- [Data Engineering](https://github.com/igorbarinov/awesome-data-engineering)
- [Streaming](https://github.com/manuzhang/awesome-streaming)
- [Apache Spark](https://github.com/awesome-spark/awesome-spark) - Apache Spark is an unified engine for large-scale data processing.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would change the comment to be simpler: - Unified engine for large-scale data processing.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another thing in the actual list is that it'd look better without spaces between a couple of the contents bullet points.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get your first comment but I'm not sure that I understand the second one. Would you care giving more explanation please ?

@sindresorhus
Copy link
Owner

includes a succinct description of the project/theme at the top (example),

Not followed.

The logo should be high-DPI, so set it to maximum half the width of the original image.

Not followed.

If you use a license badge, it should be SVG, not PNG

Not followed.

@sindresorhus
Copy link
Owner

ping @zero323

@zero323
Copy link
Author

zero323 commented May 9, 2017

I believe these are OK now @sindresorhus.

@sindresorhus
Copy link
Owner

These are not done:

includes a succinct description of the project/theme at the top (example),
Not followed.

Still not. I see no description of Apache Spark at the top of your readme.

The logo should be high-DPI, so set it to maximum half the width of the original image.
Not followed.

The image doesn't exist anymore.

@zero323 zero323 closed this May 18, 2017
@zero323
Copy link
Author

zero323 commented May 18, 2017

I don't see much interest from other contributors, and personally I have other thing on my mind, so I don't think we'll be pursuing this after all. Sorry.

@sindresorhus
Copy link
Owner

I think you're giving up too easily. It takes more work to interest contributors, and being on this list definitely helps that, but if you're not willing to put in that work, I totally understand :)

oluies added a commit to awesome-spark/awesome-spark that referenced this pull request May 24, 2017
Description of Apache Spark as per sindresorhus/awesome#895
@oluies
Copy link

oluies commented May 24, 2017

Updated Readme with an Apache Spark description, is the image OK?
@sindresorhus

@sindresorhus
Copy link
Owner

Yup. Added.

sindresorhus added a commit that referenced this pull request May 25, 2017
parinck pushed a commit to parinck/awesome that referenced this pull request Jul 23, 2017
864430457 pushed a commit to 864430457/awesome that referenced this pull request Jan 7, 2018
snippergoals added a commit to snippergoals/awesome that referenced this pull request Sep 2, 2021
highmountain12321 added a commit to highmountain12321/awesome that referenced this pull request Mar 18, 2023
johnsmith1974 added a commit to johnsmith1974/test that referenced this pull request Jul 9, 2023
architectNinja pushed a commit to architectNinja/awesome that referenced this pull request Nov 21, 2023
brisbaneg2 added a commit to brisbaneg2/awesome-cloudflare-workers that referenced this pull request Aug 10, 2024
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.

5 participants