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 2014/09/11 09:35:52 UTC

Review Request 25531: OOZIE-1936 Queuedump command should display queue information for all server.

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

Review request for oozie.


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


Repository: oozie-git


Description
-------

Queuedump command should display queue information for all server.

In AdminServlet, the arguemnt of getQueueDump() method is changed from JSONObject to JSONArray. 
Hence, V1AdminServlet#getQueueDump() is revised.
I am not sure we should keep the implementation of V1AdminServlet#getQueueDump(). 


Diffs
-----

  client/src/main/java/org/apache/oozie/client/OozieClient.java d6ff2d0 
  client/src/main/java/org/apache/oozie/client/rest/JsonTags.java 8a86bf1 
  core/src/main/java/org/apache/oozie/servlet/BaseAdminServlet.java 64d3f1f 
  core/src/main/java/org/apache/oozie/servlet/V0AdminServlet.java fd573d5 
  core/src/main/java/org/apache/oozie/servlet/V1AdminServlet.java 2383433 
  core/src/main/java/org/apache/oozie/servlet/V2AdminServlet.java adaab76 
  core/src/test/java/org/apache/oozie/service/DummyV2AdminServlet.java 0e466fc 
  core/src/test/java/org/apache/oozie/service/TestHAQueueDump.java PRE-CREATION 
  core/src/test/java/org/apache/oozie/servlet/TestV1AdminServlet.java 35568e1 
  core/src/test/java/org/apache/oozie/servlet/TestV2AdminServlet.java PRE-CREATION 

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


Testing
-------

Unit test done.
Manual test done.


Thanks,

Azrael Park