> On March 7, 2018, 8:35 p.m., Steve Moist wrote:
> > 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/diff/1/?file=1972257#file1972257line388>
> >
> >     Description says it should be info, why is it warn?

Updated the description. We just don't need these flagged as ERRORS. Very 
misleading


- Arjun


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


On March 8, 2018, 2:39 p.m., Arjun Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65953/
> -----------------------------------------------------------
> 
> (Updated March 8, 2018, 2:39 p.m.)
> 
> 
> Review request for sentry, kalyan kumar kalvagadda, Na Li, Steve Moist, 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/2/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun Mishra
> 
>

Reply via email to