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

Lenni Kuff commented on SENTRY-554:
-----------------------------------

Thanks for the fix [~colinma]. The change looks fine, but can we find a way to 
just consolidate this code (maybe in the base class or a utility class) so it 
is not duplicated in multiple places? That way we will avoid bugs like this in 
the future.

> SentryHDFSServiceProcessorFactory doesn't set the ip address correctly.
> -----------------------------------------------------------------------
>
>                 Key: SENTRY-554
>                 URL: https://issues.apache.org/jira/browse/SENTRY-554
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Colin Ma
>            Assignee: Colin Ma
>         Attachments: SENTRY-554.001.patch
>
>
> The ProcessorWrapper in SentryHDFSServiceProcessorFactory has the same 
> problem in [SENTRY-484|https://issues.apache.org/jira/browse/SENTRY-484], and 
> the sentry service can't get the ip address correctly



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

Reply via email to