-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53087/
-----------------------------------------------------------
(Updated Oct. 26, 2016, 9:32 a.m.)
Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty.
Changes
-------
Minor refactoring and unused code removal
Bugs: AMBARI-18660
https://issues.apache.org/jira/browse/AMBARI-18660
Repository: ambari
Description
-------
Log Search periodically queries the solr for the field meta data, which may not
work on some clusters. Scheduling this task should be done by spring.
Also the luke request sent for this purpose must be requested from each shard,
and the results must be summarized. Unfortunately the solr client can not be
used for this purpose because it sends it to random shard.
Schema meta data interval is also configurable now via a new custom logsearch
proprety:
logsearch.schema.fields.populate.interval.mins
Diffs (updated)
-----
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/conf/SolrConfig.java
79df3c2
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/conf/SolrPropsConfig.java
78012b0
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/conf/SolrUserPropsConfig.java
38366d5
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/converter/AbstractOperationHolderConverter.java
5945d72
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/converter/AbstractSearchRequestQueryConverter.java
2604f13
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/AuditSolrDao.java
2fbdb46
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/ServiceLogsSolrDao.java
0e1d57c
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/SolrAliasDao.java
6389446
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/SolrCollectionDao.java
c83cd99
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/SolrSchemaFieldDao.java
35bc2dc
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/UserConfigSolrDao.java
418a405
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/doc/DocConstants.java
e6f5103
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/AuditLogsManager.java
aded1d4
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ServiceLogsManager.java
44d0c00
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/rest/ServiceLogsResource.java
492a5b2
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/util/SolrUtil.java
7a856d2
Diff: https://reviews.apache.org/r/53087/diff/
Testing
-------
Done.
Thanks,
Miklos Gergely
