[jira] [Commented] (TINKERPOP-2179) Have o.a.t.g.driver.ser.SerializationException extend IOException

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on TINKERPOP-2179:
-------------------------------------------

spmallette commented on pull request #1083: TINKERPOP-2179: Have SerializationException extend IOException
URL: https://github.com/apache/tinkerpop/pull/1083
 
 
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Have o.a.t.g.driver.ser.SerializationException extend IOException
> -----------------------------------------------------------------
>
>                 Key: TINKERPOP-2179
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2179
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 3.3.5
>            Reporter: Kevin Gallardo
>            Priority: Major
>
> Suggestion to have the SerializationException from gremlin extend Java's IOException. Some other libraries like Jackson do it.
> In Java 8 there is a new {{UncheckedIOException()}} that can be created and thrown at runtime, except it requires the underlying exception to be an IOException.
> Ideally if SerializationException extended IOException we could do something like:
> {code:java}
>       try {
>         GraphBinaryWriter.write(object, bytebuf);
>       } catch (IOException e) {
>         throw new UncheckedIOException("Could not serialize", e)
>       }
> {code}
> So that the calling code does not throw a checked exception anymore. Also being able to catch IOException there would allow to be more generic.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)