You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Azrael Park <az...@nexr.com> on 2015/04/24 09:43:07 UTC

Review Request 33470: OOZIE-2163 Remove CoordinatorStore

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

Review request for oozie.


Summary (updated)
-----------------

OOZIE-2163 Remove CoordinatorStore


Bugs: OOZIE-2163
    https://issues.apache.org/jira/browse/OOZIE-2163


Repository: oozie-git


Description (updated)
-------

OOZIE-2163 Remove CoordinatorStore.
Make CoordinatorStore,CoordinatorStoreService to @Dprecated.


Diffs (updated)
-----

  core/src/main/java/org/apache/oozie/service/CoordinatorStoreService.java c941813 
  core/src/main/java/org/apache/oozie/store/CoordinatorStore.java 1bb7860 
  core/src/main/resources/oozie-default.xml ecc0d47 
  core/src/test/java/org/apache/oozie/TestCoordinatorEngine.java 8b5bee0 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordActionsIgnoreXCommand.java 2ff89b2 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java 5d779b4 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordRerunXCommand.java 53c5122 
  core/src/test/java/org/apache/oozie/command/coord/TestFutureActionsTimeOut.java 801207b 
  core/src/test/java/org/apache/oozie/command/coord/TestPastActionsTimeOut.java 708dcaf 
  core/src/test/java/org/apache/oozie/service/TestRecoveryService.java 9f8e65f 
  core/src/test/java/org/apache/oozie/store/TestCoordinatorStore.java b8b2405 
  core/src/test/java/org/apache/oozie/test/XDataTestCase.java e885892 
  core/src/test/java/org/apache/oozie/test/XTestCase.java 27cbdc9 

Diff: https://reviews.apache.org/r/33470/diff/


Testing (updated)
-------

Unit test.


Thanks,

Azrael Park


Re: Review Request 33470: OOZIE-2163 Remove CoordinatorStore

Posted by Azrael Park <az...@nexr.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33470/
-----------------------------------------------------------

(Updated April 25, 2015, 6:58 a.m.)


Review request for oozie.


Changes
-------

Addressed comments.
Removed the CoordinatorStore and CoordinatorStoreService.


Bugs: OOZIE-2163
    https://issues.apache.org/jira/browse/OOZIE-2163


Repository: oozie-git


Description
-------

OOZIE-2163 Remove CoordinatorStore.
Make CoordinatorStore,CoordinatorStoreService to @Dprecated.


Diffs (updated)
-----

  core/src/main/java/org/apache/oozie/service/CoordinatorStoreService.java c941813 
  core/src/main/java/org/apache/oozie/service/SLAStoreService.java 75e46a2 
  core/src/main/java/org/apache/oozie/service/StoreService.java bc8a2b6 
  core/src/main/java/org/apache/oozie/store/CoordinatorStore.java 1bb7860 
  core/src/main/resources/oozie-default.xml ecc0d47 
  core/src/test/java/org/apache/oozie/TestCoordinatorEngine.java 8b5bee0 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordActionsIgnoreXCommand.java 2ff89b2 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java 5d779b4 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordELExtensions.java f9a276f 
  core/src/test/java/org/apache/oozie/command/coord/TestCoordRerunXCommand.java 53c5122 
  core/src/test/java/org/apache/oozie/command/coord/TestFutureActionsTimeOut.java 801207b 
  core/src/test/java/org/apache/oozie/command/coord/TestPastActionsTimeOut.java 708dcaf 
  core/src/test/java/org/apache/oozie/service/TestRecoveryService.java 9f8e65f 
  core/src/test/java/org/apache/oozie/store/TestCoordinatorStore.java b8b2405 
  core/src/test/java/org/apache/oozie/test/XDataTestCase.java e885892 
  core/src/test/java/org/apache/oozie/test/XTestCase.java 27cbdc9 

Diff: https://reviews.apache.org/r/33470/diff/


Testing
-------

Unit test.


Thanks,

Azrael Park