PR Review: The Errors Should Be Nurtured
PR Review: The Errors Should Be Nurtured
A better way to prevent this would be to just include the actual locations in the error. It doesn’t prevent support calls, but it does make them a lot easier to solve.
Join the DZone community and get the full member experience.
Join For FreeRavenDB vs MongoDB: Which is Better? This White Paper compares the two leading NoSQL Document Databases on 9 features to find out which is the best solution for your next project.
I ran into the following in a PR that I recently reviewed. Take a look and try to figure out why I’m pointing these bits out:
![image_thumb[2]](https://ayende.com/blog/Images/Open-Live-Writer/PR-Review-the-errors-should-be-nurtured_C6C/image_thumb[2]_2.png "image_thumb[2]")
![image_thumb[3]](https://ayende.com/blog/Images/Open-Live-Writer/PR-Review-the-errors-should-be-nurtured_C6C/image_thumb[3]_2.png "image_thumb[3]")
Those are bad errors — bad in the sense that they are hiding information. Sure, in 90% of the cases, the user can just put it in the backup location or the restore location. This way, they know what the application is referring to. But the other 10% is looking at the exception from the logs, getting an “I got an error” support call and all sorts of other weird stuff.
In particular, one of the most annoying things is when the user and the application disagree on a relative path. For example, if the user believes that the relative path is based on one location and the application another, hilarity ensues. And not in a fun way.
A better way to prevent all of those would be to just include the actual locations (fully resolved, mind you) in the error. It doesn’t prevent support calls, but it does make them a lot easier to solve.
Get comfortable using NoSQL in a free, self-directed learning course provided by RavenDB. Learn to create fully-functional real-world programs on NoSQL Databases. Register today.
Like This Article? Read More From DZone
Published at DZone with permission of Oren Eini, CEO RavenDB , DZone MVB. See the original article here.
Opinions expressed by DZone contributors are their own.
{{ parent.title || parent.header.title}}
{{ parent.tldr }}
{{ parent.linkDescription }}
{{ parent.urlSource.name }}