apache / apache/accumulo

Consolidate ThriftClient code to use ServiceDescriptors

Open
#6,291 0 comments 0 reactions 1 assignee Claimed by @ddanielr View on GitHub
enhancement
Dominant language
Java
Stars
1.2k
Forks
487
Avg merge
4d 5h
Merged PRs (30d)
13

Description

**Is your feature request related to a problem? Please describe.**
The thrift client connection code uses various different methods to get connections to servers.
These various classes make troubleshooting needlessly complicated and could be simplified.

**Describe the solution you'd like**
The thrift client code should be consolidated to a single ThriftClient class which uses ServiceDescriptors to return the desired thrift connection type.

This simplified code should also implement the debug option from TServerClient
`org.apache.accumulo.client.rpc.debug.host` that allows the client to select the specific host for debugging purposes.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.