> On June 14, 2016, 8:18 a.m., Miklos Gergely wrote: > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VUserConfig.java, > > line 35 > > <https://reviews.apache.org/r/48490/diff/2/?file=1416546#file1416546line35> > > > > Why not using camel case for filter name?
wanted to be consistent with solr field name. - Dharmesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/#review137465 ----------------------------------------------------------- On June 14, 2016, 10:54 a.m., Dharmesh Makwana wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48490/ > ----------------------------------------------------------- > > (Updated June 14, 2016, 10:54 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Don Bosco > Durai, Jaimin Jetly, Oliver Szabo, Robert Nettleton, Sandor Magyari, Sumit > Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-17140 > https://issues.apache.org/jira/browse/AMBARI-17140 > > > Repository: ambari > > > Description > ------- > > Patch contains > 1) Last button in the log search pagination panel does not take the user to > the last page. > 2) Solr warming feature added. > 3) Suffix in columns mapping issue fixed. > > > Diffs > ----- > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/LogSearchConstants.java > 917956f > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/SolrDaoBase.java > 147e148 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/AuditMgr.java > 53e386e > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/LogsMgr.java > ca63671 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/MgrBase.java > 1d069d3 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/UserConfigMgr.java > 3b23f2a > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/query/QueryGeneration.java > 4647c7b > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/rest/AuditREST.java > 92bfb01 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/rest/DashboardREST.java > 1e107ed > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/util/ConfigUtil.java > 44d184d > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VUserConfig.java > 55ec1c0 > > ambari-logsearch/ambari-logsearch-portal/src/main/resources/default.properties > 25b4f1a > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/META-INF/applicationContext.xml > 6a73d60 > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/collections/BaseCollection.js > 24b6974 > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/utils/Intro.js > 869a1d4 > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/audit/AuditTabLayoutView.js > 5f93c5a > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/dialog/ApplySearchFilterView.js > 4babebc > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/tabs/EventHistoryLayoutView.js > 0f0ce58 > > Diff: https://reviews.apache.org/r/48490/diff/ > > > Testing > ------- > > Setup Logsearch on 2 node cluster and tested the above feature. > > > Thanks, > > Dharmesh Makwana > >