Iโll start a new series to record my Xvc development progress. I think itโs nice to have a dialogue between ๐ข and ๐ to explain the decisions and progress. These will be published on my personal blog for now. In the future, we can have a dedicated blog for this.
๐ LGTM. I think you can also link to commits here when they occur.
๐ข I think thatโs a bit too much work, but PRs may be linked.
๐ Ok. Letโs start with the PRs. What are you working on these days?
๐ข I started fixing warnings in issue #3 and am now fixing missing documentation warnings.
๐ Do you really think that weโll have a use for [RMNStore] in the future?
๐ข Iโm not sure. Possibly any M-N relationship can be tracked by two 1-N relationships. Under the hood, this is what RMNStore does actually.
๐ Maybe we should just remove it.
๐ข I want to save it for the time being. If [xvc-ecs] is used in other places, it may be needed.
๐ Ok. Probably YAGNI, but letโs keep it for the time being.
๐ข Writing documentation can be a debugging method, too. I think Iโve found the reason for a bug here.
๐ Um, how could it lead to a bug?
๐ข Insert means a new event happened. But when converting a VStore to an XvcStore, new events are not happening. We are simply converting from one store to another.
๐ That means they are added to XvcStore::current events, not XvcStore::previous, and when save_dir is called, they are saved.
๐ข Yeah, that was the bug. From time to time, I was seeing duplicate JSON files.
๐ Then maybe itโs better to limit all insert usage.
๐ข Yeah, I should review that, too.
๐ Added a bug report.
๐ข Iโm installing cargo-geiger to show that there is no unsafe code.
๐ Do you think youโll maintain Xvc that way? That it wonโt require any unsafe code?
๐ข Maintain the forbidden(unsafe) status? I think so. Xvc doesnโt seem to need any unsafe blocks by itself. Most of the dependencies may need it, though.
๐ Working on it, and it will be merged today.
๐ข I worked on that s3cmd bug yesterday, and it turned out that my modification of the configuration was the cause.
๐ You were thinking that itโs about R2 all along?
๐ข I should suspect my custom config more.
๐ Anyway, congratulations ๐ that Xvc now has Cloudflare R2 support.