Skip to content

[S3-data-lake] Remove/Rename all occurences of iceberg.v2 - #51012

Merged
Francis Genet (frifriSF59) merged 9 commits into
masterfrom
frifri/s3-data-lake-folder-cleanup
Jan 9, 2025
Merged

Francis Genet (frifriSF59) merged 9 commits into
masterfrom
frifri/s3-data-lake-folder-cleanup

Conversation

@frifriSF59

Copy link
Copy Markdown
Contributor

What

Update all occurrences of iceberg.v2 in our code since it should now be called s3-data-lake

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Closes https://github.com/airbytehq/airbyte-internal-issues/issues/11314

@vercel

vercel Bot commented Jan 9, 2025 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 7:34pm

@frifriSF59 Francis Genet (frifriSF59) changed the title Remove/Rename all occurences of iceberg.v2 [S3-data-lake] Remove/Rename all occurences of iceberg.v2 Jan 9, 2025
@frifriSF59

Francis Genet (frifriSF59) commented Jan 9, 2025 •

Copy link
Copy Markdown
Contributor Author

/format-fix

Format-fix job started... Check job output.

✅ Changes applied successfully. (3d3011e)

@edgao Edward Gao (edgao) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

probably also worth renaming e.g. IcebergDestination -> S3DataLakeDestination? otherwise lgtm

This branch was successfully deployed

1 active deployment
Preview — 2aba8286 Deployed Jan 9, 2025 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/destination/s3-data-lake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants