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

ASF GitHub Bot commented on TWILL-137:
--------------------------------------

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

    https://github.com/apache/incubator-twill/pull/46#discussion_r32978738
  
    --- Diff: 
twill-api/src/main/java/org/apache/twill/api/TwillRunResources.java ---
    @@ -53,4 +53,10 @@
        * @return the debug port of the container's JVM, or null if not 
debug-enabled.
        */
       Integer getDebugPort();
    +
    +  /**
    +   * @return the enabled log level for the container where the runnable is 
running in.
    +   */
    +  String getLogLevel();
    --- End diff --
    
    Would be better to return `LogEntry.Level` to make it symmetric to the 
setLogLevel method.


> Be able to get log level for each runnable via 
> TwillController.getResourceReport
> --------------------------------------------------------------------------------
>
>                 Key: TWILL-137
>                 URL: https://issues.apache.org/jira/browse/TWILL-137
>             Project: Apache Twill
>          Issue Type: Improvement
>          Components: api, core
>            Reporter: Henry Saputra
>            Assignee: Henry Saputra
>            Priority: Minor
>
> With TWILL-24 resolved, the client should be able to get the log level for an 
> application broken down by each runnable.
> This would involve adding log level to TwillRunResources interface and its 
> implementations.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to