risdenk commented on a change in pull request #61: KNOX-1793 - 
DefaultKeystoreService should not validate the signing key on initialization
URL: https://github.com/apache/knox/pull/61#discussion_r260863833
 
 

 ##########
 File path: 
gateway-server/src/main/java/org/apache/knox/gateway/services/DefaultGatewayServices.java
 ##########
 @@ -51,6 +50,33 @@
 
   private static GatewayMessages log = MessagesFactory.get( 
GatewayMessages.class );
 
+  private static final String[] SERVICE_START_ORDER = new String[]{
 
 Review comment:
   Not sure if it is easy to reverse the array. Might be easier to do 
`Arrays.asList(...)`. Your comment about reversing the list for stop makes 
sense to me.
   
   Side note - I've wanted to do this for a long time. Even on the init method. 
Init is harder since there are dependencies but still seems like it should be 
possible.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to