Skip to content

Conversation

@chenzl25
Copy link
Contributor

@chenzl25 chenzl25 commented Dec 25, 2024

- This reverts commit cda4a0c.
- The scheme of google cloud storage should be gs instead of gcs and gs is not a typo. gs is widely used by google and opendal.

  • support both gs and gcs schemes for google cloud storage

@chenzl25 chenzl25 changed the title Revert "chore: fix typo in FileIO Schemes (#653)" fix: revert "chore: fix typo in FileIO Schemes (#653)" Dec 25, 2024
@Xuanwo
Copy link
Member

Xuanwo commented Dec 25, 2024

We can support both of them

@chenzl25 chenzl25 changed the title fix: revert "chore: fix typo in FileIO Schemes (#653)" fix: support both gs and gcs schemes for google cloud storage Dec 25, 2024
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you!

@Xuanwo Xuanwo merged commit d51e818 into apache:main Dec 25, 2024
16 checks passed
chenzl25 added a commit to risingwavelabs/iceberg-rust that referenced this pull request Dec 27, 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.

2 participants