The metadata writing and snapshot writing functions both return std::expected<void, std::string>. This value should be error checked. I think correct behavior is to not abort and continue logging data, but keep the error message and maybe even the timestamps affected. Then the run function should return the error message and it should be part of the errors sent to the DB/file at the end of the run.
The metadata writing and snapshot writing functions both return std::expected<void, std::string>. This value should be error checked. I think correct behavior is to not abort and continue logging data, but keep the error message and maybe even the timestamps affected. Then the run function should return the error message and it should be part of the errors sent to the DB/file at the end of the run.