cocoindex
cocoindex copied to clipboard
[FEATURE] make target deletion (drop) tolerate failures
When we delete (drop) a target backend, sometimes it fails because the connection is no longer available. Currently this may block flow's setup process.
We want to make target deletion (drop) tolerate failures. Specifically:
- Add a new field
ignore_target_drop_failurestoSettings. Use env variableCOCOINDEX_IGNORE_TARGET_DROP_FAILURESas the default value. - When
ignore_target_drop_failuresis set totrue, when there's any error in dropping a target backend, log an error instead of popping up the error which terminates the process. - When
ignore_target_drop_failuresis set tofalse, in the error message created during dropping a target backend, put a hint to let users know about theCOCOINDEX_IGNORE_TARGET_DROP_FAILURESoption.
❤️ Contributors, please refer to 📙Contributing Guide.
Unless the PR can be sent immediately (e.g. just a few lines of code), we recommend you to leave a comment on the issue like I'm working on it or Can I work on this issue? to avoid duplicating work. Our Discord server is always open and friendly.
@georgeh0 Hi, I can take this
assigned, thanks!