Skip to content

Using raise instead, throw wraps the error#2

Open
sashman wants to merge 1 commit into
derailed:masterfrom
sashman:master
Open

Using raise instead, throw wraps the error#2
sashman wants to merge 1 commit into
derailed:masterfrom
sashman:master

Conversation

@sashman

@sashman sashman commented Feb 15, 2018

Copy link
Copy Markdown

Using throw does not reflect the original error being generated. For example:

With throw in gen_body/3 pre you get:

** (throw) %Ecto.NoResultsError{message: "expected at least one result but got none in query:\n\nfrom u in MyApp.User,\n  where: u.id == ^550\n"}

But with raise I get:

     ** (Ecto.NoResultsError) expected at least one result but got none in query:
     
     from u in MyApp.User,
       where: u.id == ^546

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant