Return-Path: X-Original-To: apmail-accumulo-notifications-archive@minotaur.apache.org Delivered-To: apmail-accumulo-notifications-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 03FB11071C for ; Fri, 23 Jan 2015 19:25:36 +0000 (UTC) Received: (qmail 92738 invoked by uid 500); 23 Jan 2015 19:25:36 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 92697 invoked by uid 500); 23 Jan 2015 19:25:36 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 92680 invoked by uid 99); 23 Jan 2015 19:25:36 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 23 Jan 2015 19:25:36 +0000 Date: Fri, 23 Jan 2015 19:25:35 +0000 (UTC) From: "Josh Elser (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (ACCUMULO-1292) Tablet constructor can hang on vfs classloader, preventing tablets from loading MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ 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)