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

Remove the Kafka to Delta table service #585

Merged
merged 2 commits into from
May 17, 2023
Merged

Conversation

DanPaseltiner
Copy link
Collaborator

PULL REQUEST

Summary

Since we are no longer using this service we are removing it before the next release.

Copy link
Member

@nickclyde nickclyde left a comment

Choose a reason for hiding this comment

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

😭

@nickclyde nickclyde changed the title Remove the Kafak to Delta table service Remove the Kafka to Delta table service May 17, 2023
@codecov
Copy link

codecov bot commented May 17, 2023

Codecov Report

Merging #585 (1c43f11) into main (52bfda6) will not change coverage.
The diff coverage is n/a.

❗ Current head 1c43f11 differs from pull request most recent head 086f4e1. Consider uploading reports for the commit 086f4e1 to get more accurate results

@@           Coverage Diff           @@
##             main     #585   +/-   ##
=======================================
  Coverage   96.42%   96.42%           
=======================================
  Files          45       45           
  Lines        2492     2492           
=======================================
  Hits         2403     2403           
  Misses         89       89           
Flag Coverage Δ
unit-tests 96.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@DanPaseltiner DanPaseltiner merged commit 122fd3b into main May 17, 2023
@DanPaseltiner DanPaseltiner deleted the remove-kafka-to-delta branch May 17, 2023 18:50
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