You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by "Andrus Adamchik (Jira)" <ji...@apache.org> on 2022/01/11 13:44:00 UTC

[jira] [Comment Edited] (CAY-2728) Add special ExtendedType to generate user-friendly exceptions

    [ https://issues.apache.org/jira/browse/CAY-2728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17472729#comment-17472729 ] 

Andrus Adamchik edited comment on CAY-2728 at 1/11/22, 1:43 PM:
----------------------------------------------------------------

In this logic we followed JPA in making "Serializable" the fallback type. So all the confusing exceptions are due to that choice. Maybe we can rethink this part?

Also I just noticed that IdGenerationMarker implements Serializable. That's really weird. I can't think of a reason why (did it leak to ROP?)



was (Author: andrus):
In this logic we followed JPA in making "Serializable" the fallback type. So all the confusing exceptions are due to that choice. Maybe we can rethink this part?

> Add special ExtendedType to generate user-friendly exceptions
> -------------------------------------------------------------
>
>                 Key: CAY-2728
>                 URL: https://issues.apache.org/jira/browse/CAY-2728
>             Project: Cayenne
>          Issue Type: Improvement
>          Components: Core Library
>    Affects Versions: 4.2.B1
>            Reporter: Nikita Timofeev
>            Assignee: Nikita Timofeev
>            Priority: Minor
>             Fix For: 4.2.B2
>
>
> We have some internal values that could be used in a DB query and cause exceptions. In that case all the information user get is a confusing message like this: "{{Haven't found suitable ExtendedType for class 'org.apache.cayenne.access.flush.IdGenerationMarker'}}". See CAY-2726 for example.
> We should add some special {{ExtendedType}} to handle those types and generate user-friendly exceptions.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)