search

earth-mover / icechunk

Open-source, cloud-native transactional tensor storage engine
https://icechunk.io
Apache License 2.0
259 stars 14 forks source link

more informative error message for failed commit #311

Open rabernat opened 2 weeks ago

rabernat commented 2 weeks ago

I intentionally created a conflict between two branches and tried to commit them both. This is the error message I got

ValueError: store error: unsuccessful repository operation: `branch update conflict: `(Some(33da463635c50b3c92a774a6)) != (Some(00666f9c21a49d0f7fa1a50d))``

Obviously this could be a lot more informative! 😆

Perhaps we just tackle this with #297, but I thought I would open an issue anyway.

paraseba commented 2 weeks ago

In general, all errors are very ugly today. If you have issues with your object store things are even uglier. We have discussed using custom exception to make them better, but we probably don't have a ticket.