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

docs: explain the value of deltalake on first page of docs #3017

Merged
merged 5 commits into from
Nov 22, 2024

Conversation

braaannigan
Copy link
Contributor

Description

The PR sets a proposed change to the text on the first page of the docs. When I arrived on these pages as a new user I found the current text did not answer the basic questions I had about the library. I've proposed a new text, though I'm happy to take on feedback.

Note: there is some overlap with the ideas on the "Why use delta lake" page. In essence I'm proposing having a quick summary of those concepts and a quickstart guide on the first page as I feel that for potential new users who have no familiarity with delta spark the current front page goes into more technical detail on that relationship than necessary.

I'd like to do a follow-on where the Why use delta lake page. I'm also wondering whether quickstart should be on a separate page in this section to make it easy to find.

Related Issue(s)

Copy link

ACTION NEEDED

delta-rs follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

@braaannigan braaannigan changed the title docs: Explain the value of deltalake on first page of docs docs: explain the value of deltalake on first page of docs Nov 21, 2024
Liam Brannigan added 5 commits November 22, 2024 10:08
Signed-off-by: Liam Brannigan <l.brannigan@analyticsengines.com>
Signed-off-by: Liam Brannigan <l.brannigan@analyticsengines.com>
Signed-off-by: Liam Brannigan <l.brannigan@analyticsengines.com>
Signed-off-by: Liam Brannigan <l.brannigan@analyticsengines.com>
Signed-off-by: Liam Brannigan <l.brannigan@analyticsengines.com>
Copy link

codecov bot commented Nov 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.48%. Comparing base (25ce389) to head (6946514).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3017      +/-   ##
==========================================
+ Coverage   72.47%   72.48%   +0.01%     
==========================================
  Files         128      128              
  Lines       40831    40831              
  Branches    40831    40831              
==========================================
+ Hits        29591    29598       +7     
- Misses       9355     9357       +2     
+ Partials     1885     1876       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@ion-elgreco ion-elgreco added this pull request to the merge queue Nov 22, 2024
Merged via the queue into delta-io:main with commit c091a82 Nov 22, 2024
24 checks passed
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.

Answer "Why use DeltaLake?" question in the docs
2 participants