Re: Review Request 52958: Fix Log Search User Config bug
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52958/ --- (Updated Nov. 7, 2016, 9:25 a.m.) Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. Changes --- fix branches Bugs: AMBARI-18616 https://issues.apache.org/jira/browse/AMBARI-18616 Repository: ambari Description --- Log Search User Config UI screen does not offer the chance to modify the filter settings for those logs which don't have any log entries loaded so far. For these logs the initial settings will be in effect until the first modification of the filters when all of these settings will be erased, and thus every log will be loaded for them without filtering. The User Config UI should always show all the logs, first those which already have some log entries in the solr, then the rest. Also some minor refactoring for cleaner code. Diffs - ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/HadoopServiceConfigHelper.java PRE-CREATION ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/UserConfigSolrDao.java 58337f7 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ManagerBase.java 45d66cf ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ServiceLogsManager.java d276769 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/common/LogFeederDataMap.java b09610c ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/filter/CreateLogfeederFilterView.js 9bdf0fa Diff: https://reviews.apache.org/r/52958/diff/ Testing --- Done Thanks, Miklos Gergely
Re: Review Request 52958: Fix Log Search User Config bug
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52958/#review155056 --- Ship it! Ship It! - Oliver Szabo On Oct. 18, 2016, 12:48 p.m., Miklos Gergely wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52958/ > --- > > (Updated Oct. 18, 2016, 12:48 p.m.) > > > Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. > > > Bugs: AMBARI-18616 > https://issues.apache.org/jira/browse/AMBARI-18616 > > > Repository: ambari > > > Description > --- > > Log Search User Config UI screen does not offer the chance to modify the > filter settings for those logs which don't have any log entries loaded so > far. For these logs the initial settings will be in effect until the first > modification of the filters when all of these settings will be erased, and > thus every log will be loaded for them without filtering. > > The User Config UI should always show all the logs, first those which already > have some log entries in the solr, then the rest. > > Also some minor refactoring for cleaner code. > > > Diffs > - > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/HadoopServiceConfigHelper.java > PRE-CREATION > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/UserConfigSolrDao.java > 58337f7 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ManagerBase.java > 45d66cf > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ServiceLogsManager.java > d276769 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/common/LogFeederDataMap.java > b09610c > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/filter/CreateLogfeederFilterView.js > 9bdf0fa > > Diff: https://reviews.apache.org/r/52958/diff/ > > > Testing > --- > > Done > > > Thanks, > > Miklos Gergely > >
Re: Review Request 52958: Fix Log Search User Config bug
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52958/#review153091 --- Ship it! Ship It! - Robert Nettleton On Oct. 18, 2016, 12:48 p.m., Miklos Gergely wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52958/ > --- > > (Updated Oct. 18, 2016, 12:48 p.m.) > > > Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. > > > Bugs: AMBARI-18616 > https://issues.apache.org/jira/browse/AMBARI-18616 > > > Repository: ambari > > > Description > --- > > Log Search User Config UI screen does not offer the chance to modify the > filter settings for those logs which don't have any log entries loaded so > far. For these logs the initial settings will be in effect until the first > modification of the filters when all of these settings will be erased, and > thus every log will be loaded for them without filtering. > > The User Config UI should always show all the logs, first those which already > have some log entries in the solr, then the rest. > > Also some minor refactoring for cleaner code. > > > Diffs > - > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/HadoopServiceConfigHelper.java > PRE-CREATION > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/UserConfigSolrDao.java > 58337f7 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ManagerBase.java > 45d66cf > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ServiceLogsManager.java > d276769 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/common/LogFeederDataMap.java > b09610c > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/filter/CreateLogfeederFilterView.js > 9bdf0fa > > Diff: https://reviews.apache.org/r/52958/diff/ > > > Testing > --- > > Done > > > Thanks, > > Miklos Gergely > >
Re: Review Request 52958: Fix Log Search User Config bug
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52958/ --- (Updated Oct. 18, 2016, 12:48 p.m.) Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. Changes --- Some more refactorings for a cleaner code Bugs: AMBARI-18616 https://issues.apache.org/jira/browse/AMBARI-18616 Repository: ambari Description --- Log Search User Config UI screen does not offer the chance to modify the filter settings for those logs which don't have any log entries loaded so far. For these logs the initial settings will be in effect until the first modification of the filters when all of these settings will be erased, and thus every log will be loaded for them without filtering. The User Config UI should always show all the logs, first those which already have some log entries in the solr, then the rest. Also some minor refactoring for cleaner code. Diffs (updated) - ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/common/HadoopServiceConfigHelper.java PRE-CREATION ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/dao/UserConfigSolrDao.java 58337f7 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ManagerBase.java 45d66cf ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/ServiceLogsManager.java d276769 ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/model/common/LogFeederDataMap.java b09610c ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/filter/CreateLogfeederFilterView.js 9bdf0fa Diff: https://reviews.apache.org/r/52958/diff/ Testing --- Done Thanks, Miklos Gergely