-
-
Notifications
You must be signed in to change notification settings - Fork 692
Panic with PrimaryKey is not set in update #2615
Copy link
Copy link
Closed
Description
There was a "PrimaryKey is not set" panic happening somewhere, and without a full backtrace environment it was hard to figure out where this was coming from. This took quite a while longer to find out why, as we expected that somehow the schema was borked based on other reports of what could cause this. Panics mess with logging machinery by truncating important output before it is shown. There was an edge case where an entry update could be called before an initial store actually succeeded, and apparently it can lead to the panic in
Line 99 in a3a47dd
| ActiveValue::NotSet => panic!("PrimaryKey is not set"), |
Edit: no, the issue was actually some freaky state issue, but I think the points still hold
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels