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

chore: remove the delta-checkpoint lambda which I have moved to a new repo #1423

Merged
merged 2 commits into from
Jun 2, 2023

Conversation

rtyler
Copy link
Member

@rtyler rtyler commented Jun 2, 2023

This can now be found in https://github.com/delta-incubator/lambdas where it can be iterated upon and released as a downstream project of deltalake crate releases

… repo

This can now be found in https://github.com/delta-incubator/lambdas where it can
be iterated upon and released as a downstream project of deltalake crate
releases
@houqp
Copy link
Member

houqp commented Jun 2, 2023

thoughts on mentioning the downstream project in the readme for awareness?

@rtyler
Copy link
Member Author

rtyler commented Jun 2, 2023

@houqp Not particularly, we had not mentioned it previously 🤷 Based on real-world experience the checkpoint lambda in its current form suffered from some memory problems in production anyways that need to be ironed out 😆

@rtyler rtyler marked this pull request as ready for review June 2, 2023 05:54
@rtyler rtyler enabled auto-merge (rebase) June 2, 2023 05:55
@rtyler rtyler merged commit 75a6e94 into delta-io:main Jun 2, 2023
@rtyler rtyler deleted the remove-lambda branch June 2, 2023 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants