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 volumes from docker compose #446

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Remove volumes from docker compose #446

merged 1 commit into from
Dec 12, 2023

Conversation

henneberger
Copy link
Contributor

@henneberger henneberger commented Dec 12, 2023

This removes persistence between docker-compose runes. The volume mapping seems to cause more confusion than help, and it may not be necessary since we don't persist flink state.

Fixes #348

Signed-off-by: Daniel Henneberger <git@danielhenneberger.com>
Copy link

codecov bot commented Dec 12, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (eb5c226) 52.12% compared to head (738365f) 52.12%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #446   +/-   ##
=========================================
  Coverage     52.12%   52.12%           
  Complexity     1160     1160           
=========================================
  Files           594      594           
  Lines         17231    17231           
  Branches       2077     2077           
=========================================
  Hits           8982     8982           
  Misses         7206     7206           
  Partials       1043     1043           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@henneberger henneberger merged commit 7b7b8b8 into main Dec 12, 2023
3 checks passed
@henneberger henneberger deleted the volumes branch December 12, 2023 00:44
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.

Don't use CREATE TABLE IF NOT EXIST
2 participants