[jira] [Commented] (SENTRY-1909) Improvements for memory usage when full path snapshot is sent from Sentry to NN

["Hadoop QA (JIRA)" <ji...@apache.org>]

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

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

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

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} all tests passed

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3212/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)