Skip to content

feat: Add Close() to destination interface and new writemode#211

Merged
yevgenypats merged 7 commits intomainfrom
destination-close
Oct 1, 2022
Merged

feat: Add Close() to destination interface and new writemode#211
yevgenypats merged 7 commits intomainfrom
destination-close

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

This is useful in cases where the destination plugin may need to close files or connections, or perform a final flush, when the sync is done.

This is useful for destinations where you need to close the file(s) or connections after a sync is done
@hermanschaaf hermanschaaf changed the title Destination close feat: Add Close() to destination interface Sep 30, 2022
@github-actions github-actions bot added the feat label Sep 30, 2022
@yevgenypats yevgenypats marked this pull request as ready for review October 1, 2022 11:45
@yevgenypats yevgenypats self-requested a review as a code owner October 1, 2022 11:45
@yevgenypats yevgenypats changed the title feat: Add Close() to destination interface feat: Add Close() to destination interface and new writemode Oct 1, 2022
@github-actions github-actions bot added feat and removed feat labels Oct 1, 2022
Copy link
Copy Markdown
Contributor

@yevgenypats yevgenypats left a comment

Choose a reason for hiding this comment

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

Nice. Also added new writemode overwrite-deletestale which will give destination plugins to delete stale data on close

@yevgenypats yevgenypats merged commit 8af6fcb into main Oct 1, 2022
@yevgenypats yevgenypats deleted the destination-close branch October 1, 2022 12:00
yevgenypats pushed a commit that referenced this pull request Oct 1, 2022
🤖 I have created a release *beep* *boop*
---


##
[0.11.4](v0.11.3...v0.11.4)
(2022-10-01)


### Features

* Add Close() to destination interface and new writemode
([#211](#211))
([8af6fcb](8af6fcb))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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