----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45979/#review128457 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/LoggingResourceProvider.java (line 113) <https://reviews.apache.org/r/45979/#comment191875> This doesnt seem to be used anywhere? I guess this function needs to be removed? - Jayush Luniya On April 11, 2016, 4:27 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45979/ > ----------------------------------------------------------- > > (Updated April 11, 2016, 4:27 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin > Jetly, Jayush Luniya, Robert Nettleton, Sumit Mohanty, Sebastian Toader, and > Yusaku Sako. > > > Bugs: AMBARI-15807 > https://issues.apache.org/jira/browse/AMBARI-15807 > > > Repository: ambari > > > Description > ------- > > Logsearch support was splitted into 3 different commits: > https://github.com/apache/ambari/tree/branch-dev-logsearch > - integrate logsearch module > - ambari server REST implementation for logsearch > - stack definition > > This one is the REST implementation. (by Robert Nettleton) > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/LoggingResourceDefinition.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java > c711bed > > ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java > 371411d > > ambari-server/src/main/java/org/apache/ambari/server/api/services/LoggingService.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java > f24da8d > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java > ca491f2 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/LoggingResourceProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/HostComponentLoggingInfo.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LogFileDefinitionInfo.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LogFileType.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LogLineResult.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LogQueryResponse.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingRequestHelper.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingRequestHelperFactory.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingRequestHelperFactoryImpl.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingRequestHelperImpl.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingSearchPropertyProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java > 63af4c4 > ambari-server/src/main/resources/key_properties.json 46a6cf9 > > ambari-server/src/test/java/org/apache/ambari/server/controller/logging/LogLineResultTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/controller/logging/LogQueryResponseTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/controller/logging/LoggingRequestHelperFactoryImplTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/controller/logging/LoggingSearchPropertyProviderTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/45979/diff/ > > > Testing > ------- > > Testing done. > > > Thanks, > > Oliver Szabo > >