> On May 12, 2016, 1:35 p.m., Robert Nettleton wrote: > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VList.java, > > line 46 > > <https://reviews.apache.org/r/47296/diff/2/?file=1381276#file1381276line46> > > > > This change is problematic, since the Ambari Integration API already > > relies on the return results from the LogSearch Server. > > > > At this point in the dev cycle, we can't be changing the API at all > > from LogSearch, or the Ambari Integration will be broken. > > > > Because this is one of the types serialized in responses to the REST > > API, this is a part of the API, and at this point in the dev cycle should > > be frozen. > > > > I'd request that this part of the patch be reverted. > > > > Thanks.
I have reverted the resultProperty changes, thanks. - Dharmesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47296/#review132892 ----------------------------------------------------------- On May 13, 2016, 6:48 a.m., Dharmesh Makwana wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47296/ > ----------------------------------------------------------- > > (Updated May 13, 2016, 6:48 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/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 > >