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

(Updated Feb. 16, 2017, 12:02 a.m.)


Review request for geode, Lynn Hughes-Godfrey, nabarun nag, and Dan Smith.


Changes
-------

modified exception handling when executing the LuceneFunction.  It will now log 
as debug level. 
Also catching PrimaryBucketException and logging afterPrimary exceptions as 
info instead of warning.


Repository: geode


Description
-------

Reduced logging level of specific exceptions from warn level to debug

I can collapse the exceptions into a single catch, wasn't sure if we wanted to 
log something different for each type or not...

No longer catching all exceptions when closing the lucene index, instead only 
handling specific ones, we can add more if we see others being thrown...


Diffs (updated)
-----

  
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneBucketListener.java
 0af2719 
  
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneEventListener.java
 44453e4 
  
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/distributed/LuceneFunction.java
 9d45526 

Diff: https://reviews.apache.org/r/56719/diff/


Testing
-------


Thanks,

Jason Huynh

Reply via email to