tinkerpop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TINKERPOP-2179) Have o.a.t.g.driver.ser.SerializationException extend IOException
Date Thu, 04 Apr 2019 11:29:00 GMT

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

ASF GitHub Bot commented on TINKERPOP-2179:

spmallette commented on pull request #1084: TINKERPOP-2179: Have SerializationException extend
IOException [master]
URL: https://github.com/apache/tinkerpop/pull/1084
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:

> 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

View raw message