Skip to content

Update acronym ETL/ELT in README.md #6500

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 5, 2025
Merged

Update acronym ETL/ELT in README.md #6500

merged 4 commits into from
Feb 5, 2025

Conversation

galenseilis
Copy link
Contributor

It is "ETL" not "ELT"

https://en.wikipedia.org/wiki/Extract,_transform,_load

Assuming I inferred the desired acronym. Feedback appreciated!

@galenseilis
Update acronym ETL/ELT in README.md
cd82505 
It is "ETL" not "ELT"

https://en.wikipedia.org/wiki/Extract,_transform,_load

Assuming I inferred the desired acronym. Feedback appreciated!
@alanmcruickshank
Copy link
Member

The acronym is intentionally ELT, referring to the paradigm of transforming after load rather than before - however we probably don't need to be that exclusive. I'd be up for including both if you think that's a good middle ground: ETL/ELT.

What do you think?

https://www.techtarget.com/searchdatamanagement/definition/Extract-Load-Transform-ELT

@galenseilis
Update README.md
f71173a 
Change back to us ELT, but with a URL explaining the difference.
@galenseilis
Copy link
Contributor Author

The acronym is intentionally ELT, referring to the paradigm of transforming after load rather than before - however we probably don't need to be that exclusive. I'd be up for including both if you think that's a good middle ground: ETL/ELT.

What do you think?

https://www.techtarget.com/searchdatamanagement/definition/Extract-Load-Transform-ELT

Ah! Thank you for that link! That was helpful in making me aware of the context.

Actually, I think ELT is fine.

I have added a commit to link to that explanation you provided. My rationale for that change is that it will help people like myself quickly understand what "ELT" is rather than "ETL". May I have some feedback on that?

alanmcruickshank
alanmcruickshank approved these changes Feb 5, 2025
View reviewed changes
Copy link
Member

@alanmcruickshank alanmcruickshank left a comment

Choose a reason for hiding this comment

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

Nice. Sorry that this is a bit slow to merge 👍

@alanmcruickshank alanmcruickshank added this pull request to the merge queue Feb 5, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Results ✅

Name    Stmts   Miss  Cover   Missing
-------------------------------------
TOTAL   19304      0   100%

248 files skipped due to complete coverage.

Merged via the queue into sqlfluff:main with commit f63d03a Feb 5, 2025
29 checks passed
@galenseilis
Copy link
Contributor Author

Coverage Results ✅

Name    Stmts   Miss  Cover   Missing
-------------------------------------
TOTAL   19304      0   100%

248 files skipped due to complete coverage.

Neat automation!

@galenseilis
Copy link
Contributor Author

Nice. Sorry that this is a bit slow to merge 👍

All good! :) It is a low-value change so I expect it to be low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alanmcruickshank alanmcruickshank alanmcruickshank approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@galenseilis @alanmcruickshank