accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3594) Consider message replacement for Writable classes internally
Date Thu, 19 Feb 2015 21:09:12 GMT

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

Josh Elser commented on ACCUMULO-3594:
--------------------------------------

I agree with you on thrift. Personally, I find PBs to "feel" a little more friendly to use,
but that's a poor technical reason. I haven't really investigated how the protoc (pb compiler)
fares in comparison to the thrift one aside from the fact that building protoc is much less
painful than thrift. Would need to look at how things fail/work across the version used to
generate the classes and library available at runtime.

Hadoop also has a version of pb that it uses for its RPC, so that's also a concern to be aware
of for the same reasons.

> Consider message replacement for Writable classes internally
> ------------------------------------------------------------
>
>                 Key: ACCUMULO-3594
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3594
>             Project: Accumulo
>          Issue Type: Sub-task
>            Reporter: Josh Elser
>             Fix For: 1.7.0
>
>
> We have a number of writable classes, some of which are non-trivial and serialized in
tables or files:
> * {{org.apache.accumulo.core.tabletserver.log.LogEntry}}
> * {{org.apache.accumulo.core.metadata.schema.DataFileValue}}
> * {{org.apache.accumulo.server.master.state.TServerInstance}}
> Consider converting this (and possibly others) to better message classes to easily add
more functionality in the future.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message