[ 
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)

Reply via email to