-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65953/#review198819
-----------------------------------------------------------




sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/NotificationProcessor.java
Line 388 (original), 388 (patched)
<https://reviews.apache.org/r/65953/#comment279060>

    Description says it should be info, why is it warn?


- Steve Moist


On March 7, 2018, 5:03 p.m., Arjun Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65953/
> -----------------------------------------------------------
> 
> (Updated March 7, 2018, 5:03 p.m.)
> 
> 
> Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> "Alter table event has incomplete information" and "Alter table notification 
> ignored as neither name nor location has changed" are logged as ERROR. This 
> can flood the logs and be a distraction to analyzing actual issues. These 
> should be set at INFO level
> 
> 
> Diffs
> -----
> 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/NotificationProcessor.java
>  94a0b0f12 
> 
> 
> Diff: https://reviews.apache.org/r/65953/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun Mishra
> 
>

Reply via email to