----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21880/#review43881 -----------------------------------------------------------
Ship it! Ship It! - Vamsee Yarlagadda On May 23, 2014, 10:47 p.m., Gregory Chanan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21880/ > ----------------------------------------------------------- > > (Updated May 23, 2014, 10:47 p.m.) > > > Review request for sentry and Vamsee Yarlagadda. > > > Repository: sentry > > > Description > ------- > > I've seen cases where the tests fail because it runs out memory to allocate > the cache, e.g.: > > 2014-05-23 15:20:50,236 (qtp1580548625-152) [ERROR - > org.apache.solr.core.CoreContainer.recordAndThrow(\ > CoreContainer.java:910)] Unable to create core: > allRolesCollection_shard1_replica1 > org.apache.solr.common.SolrException: The max direct memory is likely too > low. Either increase it (by \ > adding -XX:MaxDirectMemorySize=<size>g -XX:+UseLargePages to your containers > startup args) or disable d\ > irect allocation using solr.hdfs.blockcache.direct.memory.allocation=false in > solrconfig.xml. If you ar\ > e putting the block cache on the heap, your java heap size might not be large > enough. Failed allocating\ > ~134.217728 MB. > at org.apache.solr.core.SolrCore.<init>(SolrCore.java:858) > at org.apache.solr.core.SolrCore.<init>(SolrCore.java:641) > at org.apache.solr.core.CoreContainer.create(CoreContainer.java:556) > at > org.apache.solr.handler.admin.CoreAdminHandler.handleCreateAction(CoreAdminHandler.java:571) > at > org.apache.solr.handler.admin.CoreAdminHandler.handleRequestInternal(CoreAdminHandler.java:2\ > 00) > at > org.apache.solr.handler.admin.CoreAdminHandler.handleRequestBody(CoreAdminHandler.java:186) > at > org.apache.solr.handler.admin.SecureCoreAdminHandler.handleRequestBody(SecureCoreAdminHandle\ > r.java:150) > > It's probably better to just disable to cache because that isn't really what > we are testing. > > > Diffs > ----- > > > sentry-tests/sentry-tests-solr/src/test/java/org/apache/sentry/tests/e2e/solr/AbstractSolrSentryTestBase.java > b5b7678 > > Diff: https://reviews.apache.org/r/21880/diff/ > > > Testing > ------- > > > Thanks, > > Gregory Chanan > >
