The error handling in `merge` was incomplete, aka it could leave the merger in a half-modified state in case of an error. That's generally a bad idea and can lead to ugly bugs. Also the "builder" pattern that is used here usually consumes itself (and provides a clone impl), so it is easier to reason about modifications. So this commit just changes it to self-consuming builder. A nice side effect of the new pattern is also that it is build-time checked and does not contain a runtime assert any longer. |
||
---|---|---|
.. | ||
src | ||
Cargo.toml | ||
README.md |
README.md
Internal Types
This crate contains InfluxDB IOx "internal" types which are shared across crates and internally between IOx instances, but not exposed externally to clients
Internal in this case means that changing the structs is designed not to require additional coordination / organization with clients.