Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Thu, 19 Feb 2015 21:05:11 +0000 (UTC)
From: "Christopher Tubbs (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12774988.1423852902000.89577.1424379911963@Atlassian.JIRA>
In-Reply-To: <JIRA.12774988.1423852902000@Atlassian.JIRA>
References: <JIRA.12774988.1423852902000@Atlassian.JIRA>
 <JIRA.12774988.1423852902040@arcas>
Subject: [jira] [Commented] (ACCUMULO-3594) Consider message replacement for
 Writable classes internally
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Christopher Tubbs commented on ACCUMULO-3594:
---------------------------------------------

I have not tried avro, but I've been a little disillusioned by thrift's lack of regression testing (like the one inspiring THRIFT-2173) and some questionable decisions (like the one identified in ACCUMULO-2950). With both thrift and protocol buffers, there's also the frustration of performing pre-build steps for code generation which is a bit unfriendly (especially in maven, and especially when reliant on a particular version of the compiler).

However, I don't have a strong preference.

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