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

Hadoop QA commented on SENTRY-1909:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12884605/SENTRY-1909.01.patch 
against master.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/3202/console

This message is automatically generated.

> Improvements for memory usage when full path snapshot is sent from Sentry to 
> NN
> -------------------------------------------------------------------------------
>
>                 Key: SENTRY-1909
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1909
>             Project: Sentry
>          Issue Type: Improvement
>          Components: Sentry
>    Affects Versions: 2.0.0
>            Reporter: Alexander Kolbasov
>            Assignee: Alexander Kolbasov
>         Attachments: SENTRY-1909.01.patch
>
>
> While looking at SENTRY-1907 I noticed another thing. 
> {{sentryStore.retrieveFullPathsImage()}} uses {{Map<String, Set<String>>}} as 
> a snapshot representation. This isn't needed since {{Map<String, 
> Collection<String>>}} is sufficient and we can represent it using ArrayList. 
> This is much more efficient representation. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to