ofuks commented on a change in pull request #842:
URL: https://github.com/apache/incubator-dlab/pull/842#discussion_r465795426



##########
File path: 
services/dlab-webapp-common/src/main/java/com/epam/dlab/rest/contracts/ComputationalAPI.java
##########
@@ -37,4 +37,5 @@
        String COMPUTATIONAL_TERMINATE_CLOUD_SPECIFIC = COMPUTATIONAL_TERMINATE 
+ "/cloud";
        String COMPUTATIONAL_LIB_INSTALL = LIBRARY + COMPUTATIONAL + 
"/lib_install";
        String COMPUTATIONAL_LIB_LIST = LIBRARY + COMPUTATIONAL + "/lib_list";
+       String COMPUTATIONAL_RECONFIGURE_MESSAGE = "Reconfigure compute <%s>, 
requested for notebook <%s>";

Review comment:
       Please rename the field to start with 'AUDIT_'.
   Also, please move the field to line 24 (under the **AUDIT_MESSAGE** 
constant), so that all audit messages are in one place




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@dlab.apache.org
For additional commands, e-mail: dev-h...@dlab.apache.org

Reply via email to