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

Add Shaped AI tap-iceberg #1808

Merged
merged 2 commits into from
Jul 18, 2024
Merged

Conversation

BTheunissen
Copy link
Contributor

Adds an extractor for Apache Iceberg data. Currently this is a WIP implementation, currently passing all integration tests for local test data, with the aim to support a variety of Iceberg catalog types such as AWS Glue, and supporting features such as incremental replication and delete row propagation.

Copy link

netlify bot commented Jul 18, 2024

Deploy Preview for meltano-hub ready!

Name Link
🔨 Latest commit 901c05c
🔍 Latest deploy log https://app.netlify.com/sites/meltano-hub/deploys/6699468c6130540008b1b819
😎 Deploy Preview https://deploy-preview-1808--meltano-hub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@edgarrmondragon
Copy link
Collaborator

Nice!

@edgarrmondragon
Copy link
Collaborator

The build is currently failing because of the missing logo file:

Error: Cannot find module './extractors/iceberg.png'

https://app.netlify.com/sites/meltano-hub/deploys/669940f9ae3ba20008e3509a

@BTheunissen BTheunissen requested a review from cjohnhanson as a code owner July 18, 2024 16:44
Copy link
Collaborator

@edgarrmondragon edgarrmondragon left a comment

Choose a reason for hiding this comment

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

Thanks @BTheunissen!

@edgarrmondragon edgarrmondragon merged commit 345f741 into meltano:main Jul 18, 2024
13 checks passed
@BTheunissen BTheunissen deleted the shaped-tap-iceberg branch July 18, 2024 17:38
@BTheunissen
Copy link
Contributor Author

No worries @edgarrmondragon thanks for the quick merge. Now to import this into our project to test, and add some more functionality to the tap!

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.

3 participants