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

NEW Logo #8

Closed
wants to merge 6 commits into from
Closed

Conversation

pinarbayata
Copy link

@pinarbayata pinarbayata commented Feb 28, 2024

Which issue does this PR close?

Closes apache#8788.

Rationale for this change

The existing logo, which solely contained DataFusion, lacked the Apache branding, which is crucial for alignment and recognition within the ASF community. Following extensive discussions and community input across three rounds of suggestions, we have settled on a new logo.

What changes are included in this PR?

Update of all instances of the old DataFusion logo across website, documentation, and marketing materials to the new Apache DataFusion logo.

Also I have added a brand guideline following Apache Foundation Style style. It can be found in ./docs/logo/Datafusion_Branding_Guideline.pdf.

Are these changes tested?

  • Engaged with the DataFusion community to gather feedback on the updated branding, ensuring it resonates well and maintains the project's identity.
  • Reviewed all updated materials for adherence to both the new branding guidelines and ASF branding standards.

Are there any user-facing changes?

N/A

@mustafasrepo
Copy link
Collaborator

Pınar ./docs/logos/source/_static/images/ directory has images also. Please add logo images that will be used in the project page there also.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 28, 2024
Copy link
Collaborator

@mustafasrepo mustafasrepo left a comment

Choose a reason for hiding this comment

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

Looking good to me!

@mustafasrepo mustafasrepo added the final_review Ozan can review this pr label Feb 28, 2024
@mustafasrepo
Copy link
Collaborator

Upstream PR

@mustafasrepo
Copy link
Collaborator

Merged upstream in PR9385.

mustafasrepo pushed a commit that referenced this pull request May 22, 2024
…he#10527)

* chore: merge main and resolve conflict

* chore: use less copy

* chore: remove clone

* remove more clones (#8)

* refactor: use HashSet<&Expr> instead of HashSet<String>

* refactor: remove more cloning

* chore: reduce string allocation

Co-authored-by: Adam Curtis <adam.curtis.dev@gmail.com>

* chore: return internal error instead of panacing

* chore: use arg display_name as hash key instead of a hashed value

---------

Co-authored-by: Adam Curtis <adam.curtis.dev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation final_review Ozan can review this pr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants