You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Oliver Szabo <os...@hortonworks.com> on 2016/04/09 13:23:52 UTC
Review Request 45979: AMBARI-15679: REST API for Logsearch

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45979/
-----------------------------------------------------------

Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Robert Nettleton, Sumit Mohanty, Sebastian Toader, and Yusaku Sako.


Bugs: AMBARI-15679
    https://issues.apache.org/jira/browse/AMBARI-15679


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