----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54339/#review157993 -----------------------------------------------------------
Ship it! Ship It! - Miklos Gergely On Dec. 5, 2016, 2:10 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54339/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2016, 2:10 p.m.) > > > Review request for Ambari, Miklos Gergely and Robert Nettleton. > > > Bugs: AMBARI-19075 > https://issues.apache.org/jira/browse/AMBARI-19075 > > > Repository: ambari > > > Description > ------- > > add cookie management for logsearch integration code. > > also contains a few fixes: > - inactive session timeout is in seconds, not in minutes > - enable info logging in log files (hard to find/debug issues if there is any > problem with logsearch portal) > > > Diffs > ----- > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/LogSearch.java > 2c3f4f5 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/web/listener/LogSearchSessionListener.java > PRE-CREATION > ambari-logsearch/docker/test-config/logsearch/log4j.xml b80824b > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingCookieStore.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingRequestHelperImpl.java > eab0c04 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-log4j.xml.j2 > ce39030 > > Diff: https://reviews.apache.org/r/54339/diff/ > > > Testing > ------- > > testing done. (FT: manually, UT: one test failing but unrelated with this > change) > > > Thanks, > > Oliver Szabo > >
