accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3855) ProxyServer.update fails silently if batchwriter doesn't exist
Date Thu, 28 May 2015 18:44:17 GMT


Josh Elser commented on ACCUMULO-3855:

Eric clarified to me that the comment means to inform the reader than the implementation can't
throw an exception because it's a one-way thrift call.

The proxy interface is intended to mimic the TabletServer's thrift methods that the batchscanner
uses. There may not be something better to do here then at the low level, but try to expose
something "nicer" to use at a slightly higher level.

> ProxyServer.update fails silently if batchwriter doesn't exist
> --------------------------------------------------------------
>                 Key: ACCUMULO-3855
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: proxy
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>            Priority: Critical
>             Fix For: 1.5.3, 1.6.3, 1.8.0, 1.7.1
> Inspecting ProxyServer, saw the following:
> {code}
>   @Override
>   public void update(String writer, Map<ByteBuffer,List<ColumnUpdate>> cells)
throws TException {
>     try {
>       BatchWriterPlusException bwpe = getWriter(writer);
>       addCellsToWriter(cells, bwpe);
>     } catch (UnknownWriter e) {
>       // just drop it, this is a oneway thrift call and throwing a TException seems to
make all subsequent thrift calls fail
>     }
>   }
> {code}
> This means that calls to update which fail to get a BatchWriter (for whatever reason)
will silently fail and appear to the user to have written data successfully. This is horrible.
> I'm not sure what the comment means either. I'm not sure why throwing a TException from
the server would invalidate the connection, but perhaps I'm just ignorant of some finer details.

This message was sent by Atlassian JIRA

View raw message