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

(Updated June 21, 2016, 2:16 p.m.)


Review request for Ambari, Don Bosco Durai, Dharmesh Makwana, Oliver Szabo, 
Robert Nettleton, and Sumit Mohanty.


Changes
-------

Updated to match latest trunk


Bugs: Ambari-17277
    https://issues.apache.org/jira/browse/Ambari-17277


Repository: ambari


Description
-------

After Log Search is started it sends the filters to the Solr which describes 
what log levels should be persisted. In the meantime Logfeeders start to push 
their data, which is not filtered. The result is misleading, the user may find 
it add that some specific level logs are loaded, others are not.


Diffs (updated)
-----

  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/LogFeeder.java
 166c0f3 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/logconfig/FetchConfigFromSolr.java
 f2d074a 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/logconfig/LogFeederConstants.java
 f61dc1b 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/output/OutputSolr.java
 a7202e7 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/util/SolrUtil.java
 29feef7 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/main/resources/logfeeder.properties
 6cba826 
  
ambari-logsearch/ambari-logsearch-logfeeder/src/test/java/org/apache/ambari/logfeeder/output/OutputSolrTest.java
 afbccca 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/templates/logfeeder.properties.j2
 9887e76 

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


Testing
-------

logfeeder:
Tests run: 35, Failures: 0, Errors: 0, Skipped: 0

ambari-server
Total run:1062
Total errors:0
Total failures:0


Thanks,

Miklos Gergely

Reply via email to