[ 
https://issues.apache.org/jira/browse/BROOKLYN-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14086220#comment-14086220
 ] 

ASF GitHub Bot commented on BROOKLYN-12:
----------------------------------------

Github user aledsage commented on a diff in the pull request:

    https://github.com/apache/incubator-brooklyn/pull/94#discussion_r15809988
  
    --- Diff: 
software/webapp/src/main/java/brooklyn/entity/webapp/JavaWebAppSoftwareProcessImpl.java
 ---
    @@ -178,33 +176,4 @@ protected void doStop() {
             setAttribute(REQUESTS_PER_SECOND_LAST, 0D);
             setAttribute(REQUESTS_PER_SECOND_IN_WINDOW, 0D);
         }
    -    
    -    protected Set<String> getEnabledProtocols() {
    -        return getAttribute(JavaWebAppSoftwareProcess.ENABLED_PROTOCOLS);
    -    }
    -    
    -    protected boolean isProtocolEnabled(String protocol) {
    -        for (String contender : getEnabledProtocols()) {
    -            if (protocol.equalsIgnoreCase(contender)) {
    -                return true;
    -            }
    -        }
    -        return false;
    -    }
    -    
    -    protected String inferBrooklynAccessibleRootUrl() {
    --- End diff --
    
    Agree to moving this code to somewhere that's not java specific (i.e. not 
in `JavaWebAppSoftwareProcessImpl`). However, we should probably deprecate this 
method and delete after releasing 0.7.


> Add Node.JS application support
> -------------------------------
>
>                 Key: BROOKLYN-12
>                 URL: https://issues.apache.org/jira/browse/BROOKLYN-12
>             Project: Brooklyn
>          Issue Type: New Feature
>         Environment: Linux
>            Reporter: Andrew Kennedy
>            Assignee: Andrew Kennedy
>
> Create entities and blueprints to deploy and manage Node.JS applications and 
> clustered services.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to