[
https://issues.apache.org/jira/browse/ACCUMULO-1292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14289784#comment-14289784
]
Josh Elser commented on ACCUMULO-1292:
--------------------------------------
I wanted to say that I'm a little frustrated that this got committed without
really responding about my concerns on test coverage (thank you, Dave. I know
you did engage me on the subject). Even if it was infeasible to add tests to
cover these changes, it would have been nice to say so rather than pretty much
ignoring the concerns and moving forward.
Also, the final patch included a guava class which doesn't exist as a
dependency in accumulo-start. I'm removing the usage of it to remove the
necessity to add a new dependency here.
> Tablet constructor can hang on vfs classloader, preventing tablets from
> loading
> -------------------------------------------------------------------------------
>
> Key: ACCUMULO-1292
> URL: https://issues.apache.org/jira/browse/ACCUMULO-1292
> Project: Accumulo
> Issue Type: Bug
> Components: tserver
> Affects Versions: 1.5.0, 1.6.0, 1.6.1
> Reporter: John Vines
> Assignee: Eric Newton
> Fix For: 1.6.2, 1.7.0
>
> Attachments: ACCUMULO-1292-04.patch, ACCUMULO-1292-05.patch,
> ACCUMULO-1292-06.patch, ACCUMULO-1292-atomic-update.patch,
> ACCUMULO-1292-using-locks.patch, ACCUMULO-1292.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Taken from TODO from r1424106 regarding ACCUMULO-867. This is something that
> we should at least look into more before 1.5 is released.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
