accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-1691) Update thrift to 0.9.1 / clean up TServerUtils
Date Fri, 06 Sep 2013 01:46:52 GMT


Christopher Tubbs commented on ACCUMULO-1691:

I ran into a problem updating thrift to 0.9.1. It seems that in TServerUtils, we extend THsHaServer
so we can override the Invocation class, for the sole purpose of setting the clientAddress
with the socket info from the frameBuffer's transport.

This actually seems completely unnecessary, because we can get this information from our ClientInfoProcessorFactory.
However, we don't seem to be using that at all... as it is only called in the TServerUtils.startThreadPoolServer,
which doesn't appear to ever be called.

It feels like we're shooting ourselves in the foot with some of this TServerUtils stuff.
> Update thrift to 0.9.1 / clean up TServerUtils
> ----------------------------------------------
>                 Key: ACCUMULO-1691
>                 URL:
>             Project: Accumulo
>          Issue Type: Task
>            Reporter: Christopher Tubbs
>            Assignee: Christopher Tubbs
>            Priority: Minor
>             Fix For: 1.6.0
> Updating to thrift 0.9.1 fixes some bugs in code generation, among other things.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message