accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ivan Bella (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4169) TabletServer.config contextCleaner removes contexts that are not set on a table
Date Wed, 23 Mar 2016 18:14:25 GMT

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

Ivan Bella commented on ACCUMULO-4169:
--------------------------------------

bq. "in use" for a context would, I think, mean the call to Context.getClassLoader() by IteratorUtil.loadIterators
or something else. You are not suggesting that we track the use of the actual classloader
that is returned from the context are you?

By "in use" I means the class loader for that context being used in a scan.  So we just need
a means to determine if there are any currently running scans using a particular context,
or get all of the contexts being used by current scans.  The routine that periodically checks
for unused contexts would have to keep track of the contexts and the last time they appears
to be used.  I think this logic should be fairly simple.

bq. Considering this, Dave's suggestion seems more reasonable to me (the lifetime of a context
is defined by the presence of the configuration property). Is there a concern in taking that
approach?

The concern I have is that each class loader takes up perm-gen space.  So cleaning them out
if not being used seems prudent.

> TabletServer.config contextCleaner removes contexts that are not set on a table
> -------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-4169
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4169
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.8.0
>            Reporter: Dave Marion
>
> ACCUMULO-3948 added a feature where you could define a context in the Accumulo configuration,
not set it on a table, and use it in a Scanner. However, there is a runnable created n TabletServer.config()
that runs every 60 seconds that closes context that are not defined on a table. Suggesting
that we have the context cleaner not close any context defined in the configuration.



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

Mime
View raw message