accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3928) TableNotFoundException triggered by MemoryManager
Date Thu, 02 Jul 2015 20:28:05 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-3928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14612467#comment-14612467
] 

Josh Elser commented on ACCUMULO-3928:
--------------------------------------

Thanks for the review, Keith!

bq. Like if a Runtime exception occurs and the table no longer exist, then ignore it.

That's a good point. I didn't make the change though. The fact that we're wrapping a checked
exception in the first place is a bit of a smell to me. It was totally obtuse to us in the
first place that the call might throw an Exception because the table is gone which hid this
bug to being with.

I think if we start having more problems, it might be a sign that we need to pass around the
exceptions a bit better.

> TableNotFoundException triggered by MemoryManager
> -------------------------------------------------
>
>                 Key: ACCUMULO-3928
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3928
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.6.4, 1.7.1, 1.8.0
>
>         Attachments: ACCUMULO-3928.1.patch
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Noticed during randomwalk on 1.8.0-SNAPSHOT.
> {noformat}
> Memory manager failed org.apache.accumulo.core.client.TableNotFoundException: Table (Id=1q5)
does not exist
> 	java.lang.IllegalArgumentException: org.apache.accumulo.core.client.TableNotFoundException:
Table (Id=1q5) does not exist
> 		at org.apache.accumulo.core.client.impl.Tables.getNamespaceId(Tables.java:239)
> 		at org.apache.accumulo.server.conf.TableParentConfiguration.getNamespaceId(TableParentConfiguration.java:38)
> 		at org.apache.accumulo.server.conf.NamespaceConfiguration.getPath(NamespaceConfiguration.java:88)
> 		at org.apache.accumulo.server.conf.NamespaceConfiguration.get(NamespaceConfiguration.java:101)
> 		at org.apache.accumulo.server.conf.ZooCachePropertyAccessor.get(ZooCachePropertyAccessor.java:110)
> 		at org.apache.accumulo.server.conf.TableConfiguration.get(TableConfiguration.java:99)
> 		at org.apache.accumulo.core.conf.AccumuloConfiguration.getTimeInMillis(AccumuloConfiguration.java:252)
> 		at org.apache.accumulo.server.tabletserver.LargestFirstMemoryManager.getMinCIdleThreshold(LargestFirstMemoryManager.java:144)
> 		at org.apache.accumulo.server.tabletserver.LargestFirstMemoryManager.getMemoryManagementActions(LargestFirstMemoryManager.java:188)
> 		at org.apache.accumulo.tserver.TabletServerResourceManager$MemoryManagementFramework.manageMemory(TabletServerResourceManager.java:408)
> 		at org.apache.accumulo.tserver.TabletServerResourceManager$MemoryManagementFramework.access$400(TabletServerResourceManager.java:318)
> 		at org.apache.accumulo.tserver.TabletServerResourceManager$MemoryManagementFramework$2.run(TabletServerResourceManager.java:346)
> 		at org.apache.accumulo.fate.util.LoggingRunnable.run(LoggingRunnable.java:35)
> 		at java.lang.Thread.run(Thread.java:745)
> 	Caused by: org.apache.accumulo.core.client.TableNotFoundException: Table (Id=1q5) does
not exist
> 		... 14 more
> {noformat}
> Marked fixVersion for all versions, under the assumption that it does affect all. Need
to verify. Should catch this case and fail gracefully, making sure that this is an expected
state to even arrive in.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message