accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Created] (ACCUMULO-2867) Consolidate Thrift connection helper classes
Date Fri, 06 Jun 2014 17:28:04 GMT
Josh Elser created ACCUMULO-2867:

             Summary: Consolidate Thrift connection helper classes
                 Key: ACCUMULO-2867
             Project: Accumulo
          Issue Type: Improvement
          Components: client, master, replication, tserver
            Reporter: Josh Elser
            Assignee: Josh Elser
            Priority: Minor

Came up during the review of ACCUMULO-378:

ReplicationClient, MasterClient, and ThriftUtil are all classes with lots of static methods
used to make executing RPC against a remote service much easier with nice constructs (like
getting a typed return object).

The downside is that they have lots of implicit gotchas contained (implicit retries, network
failure handling, etc), which make them easier to use by the invoking code, but very complex

It would be good to consolidate what we have into a common base, and then provide strong semantics
on each operation that encapsulates how we want each to operate (master ops should retry quickly,
replication ops should retry slowly and give up eventually, etc), and also allow for testing
of these intended implementations.

This message was sent by Atlassian JIRA

View raw message