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: fix csv filename in S3 bucket URI #66

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aziraphail
Copy link

Description

This PR fixes a documentation error in the README where the S3 bucket URI for the items table is incorrectly listed as raw_order_items.csv when the actual file in the bucket is raw_items.csv.

Problem

Users following the quickstart guide encounter errors when trying to load raw_order_items.csv as this file doesn't exist in the S3 bucket.

Solution

Updated the README to reference the correct filename raw_items.csv which is the actual file present in the bucket.

Verification

Verified by attempting to download both files:

  • 's3://jaffle-shop-raw/raw_order_items.csv' - fails with 403 error
  • 's3://jaffle-shop-raw/raw_items.csv' - downloads successfully

The S3 bucket contains 'raw_items.csv' rather than 'raw_order_items.csv'. This fixes the documentation to match the actual filename.
@aziraphail
Copy link
Author

@dave-connors-3 Hi! Would you have time to review this PR when convenient? Happy to address any feedback. Thank you :)

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.

1 participant