[
https://issues.apache.org/jira/browse/ACCUMULO-3856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14561703#comment-14561703
]
Josh Elser commented on ACCUMULO-3856:
--------------------------------------
(11f108e252f8358a3ac8b79843f1ebd77bee647e should be tagged for ACCUMULO-3855,
not this issue)
> ProxyServer.update fails silently if batchwriter doesn't exist
> --------------------------------------------------------------
>
> Key: ACCUMULO-3856
> URL: https://issues.apache.org/jira/browse/ACCUMULO-3856
> Project: Accumulo
> Issue Type: Bug
> Components: proxy
> Reporter: Josh Elser
> Priority: Critical
> Fix For: 1.6.3, 1.8.0, 1.7.1
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> 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
(v6.3.4#6332)
