FSA b/17116680: Change log level for unresolvable SIDs to FINEST

FSA b/17116680: Change log level for unresolvable SIDs to FINEST

[pjo, 2014-08-21 21:29:37]

LGTM.  Thank you


- technology's compounding interest
-


On Thu, Aug 21, 2014 at 1:41 PM, <Brett.Michael.Johnson@gmail.com> wrote:

> Reviewers: pjo, mifern,
>
> Description:
> FSA b/17116680: Change log level for unresolvable SIDs to FINEST
>
> Please review this at http://codereview.appspot.com/133730043/
>
> Affected files (+1, -1 lines):
>   M src/com/google/enterprise/adaptor/fs/WindowsAclFileAttributeViews.java
>
>
> Index: src/com/google/enterprise/adaptor/fs/WindowsAclFileAttributeViews.
> java
> diff --git a/src/com/google/enterprise/adaptor/fs/
> WindowsAclFileAttributeViews.java b/src/com/google/enterprise/adaptor/fs/
> WindowsAclFileAttributeViews.java
> index 66cf25e0c9afbb84c5044df3ccced2e5a1724f65..
> f92c326322bc2ced4df707c97491c944862eb32a 100644
> --- a/src/com/google/enterprise/adaptor/fs/WindowsAclFileAttributeViews.
> java
> +++ b/src/com/google/enterprise/adaptor/fs/WindowsAclFileAttributeViews.
> java
> @@ -331,7 +331,7 @@ class WindowsAclFileAttributeViews {
>      } catch (Win32Exception e) {
>        // Only the least significant 16-bits signifies the HR code.
>        if ((e.getHR().intValue() & 0xFFFF) == WinError.ERROR_NONE_MAPPED) {
> -        log.log(Level.WARNING, "Skipping ACE with unresolvable SID: {0}.",
> +        log.log(Level.FINEST, "Skipping ACE with unresolvable SID: {0}.",
>              ace.getSidString());
>          return null;
>        } else {
>
>
>

[Brett, 2014-08-25 20:46:45]

Committed 25 August 2014 to Filesystem Adaptor:

To https://code.google.com/p/plexi.fs/
   9aa177d..7019203  master -> master