----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47296/#review132928 -----------------------------------------------------------
ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VList.java <https://reviews.apache.org/r/47296/#comment197195> Any reason this is being removed? - Sumit Mohanty On May 12, 2016, 11:31 a.m., Dharmesh Makwana wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47296/ > ----------------------------------------------------------- > > (Updated May 12, 2016, 11:31 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Don Bosco > Durai, Oliver Szabo, Robert Nettleton, Sandor Magyari, Sumit Mohanty, and > Sebastian Toader. > > > Bugs: AMBARI-16629 > https://issues.apache.org/jira/browse/AMBARI-16629 > > > Repository: ambari > > > Description > ------- > > Patch contains > 1) Authentication : change the API as per suggestion and support for roles in > authentication process. > 2) resultSize property removed from collection class. > 3) "Preview" issue fixed. > > > Diffs > ----- > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/LogsMgr.java > 0442cf9 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/manager/UserConfigMgr.java > a60402e > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/query/QueryGenerationBase.java > cc61127 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/util/JSONUtil.java > 8535039 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VList.java > 97226d2 > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/web/security/LogsearchExternalServerAuthenticationProvider.java > 79a414c > > ambari-logsearch/ambari-logsearch-portal/src/main/resources/logsearch.properties > 315d736 > > Diff: https://reviews.apache.org/r/47296/diff/ > > > Testing > ------- > > Tested in my local Ambari environment. > > > Thanks, > > Dharmesh Makwana > >