[PATCH] 138 - Add Exceptions to Profile REST service
Adds exceptions to the Profile REST interface. Please review. Thanks, Ade
On 7/29/2013 10:38 PM, Ade Lee wrote:
Adds exceptions to the Profile REST interface.
Please review. Thanks, Ade
Just one thing, the error message is logged into debug log using CMS.debug(message), but the stack trace is displayed on standard output using e.printStackTrace(). Should we use CMS.debug(e) too? Other than that, ACK. -- Endi S. Dewata
acked by endi. Pushed to master. On Tue, 2013-07-30 at 11:27 -0500, Endi Sukma Dewata wrote:
On 7/29/2013 10:38 PM, Ade Lee wrote:
Adds exceptions to the Profile REST interface.
Please review. Thanks, Ade
Just one thing, the error message is logged into debug log using CMS.debug(message), but the stack trace is displayed on standard output using e.printStackTrace(). Should we use CMS.debug(e) too?
Other than that, ACK.
participants (2)
-
Ade Lee -
Endi Sukma Dewata