----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56274/ -----------------------------------------------------------
(Updated Feb. 3, 2017, 10:17 a.m.) Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs: AMBARI-19863 https://issues.apache.org/jira/browse/AMBARI-19863 Repository: ambari Description ------- There are several bugs relating to the user configs: - due to a bug saving filters do not work - saving / viewing / deleting filters uses the user name received, instead of the logged in user - deleting filter button is not visible also removed some unused code Diffs (updated) ----- ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/LogSearchConstants.java 1e46ae9 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/converter/UserConfigRequestQueryConverter.java 082f45c ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/doc/DocConstants.java caf0636 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/UserConfigManager.java 03535b1 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/request/UserConfigParamDefinition.java 23b350a ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/request/impl/UserConfigRequest.java c99ea75 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/rest/UserConfigResource.java 31f584d ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/solr/SolrConstants.java 85554bc ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/common/EventHistoryLayout.js 89914ea ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/dialog/ApplySearchFilterView.js 16a37ad ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/tabs/EventHistoryLayoutView.js b30dd50 ambari-logsearch/ambari-logsearch-portal/src/test/java/org/apache/ambari/logsearch/converter/UserConfigRequestQueryConverterTest.java 66c172a Diff: https://reviews.apache.org/r/56274/diff/ Testing ------- Tested on local cluster Thanks, Miklos Gergely