[GitHub] twill issue #14: TWILL-138 Runtime log level change

2016-12-12 Thread hsaputra
Github user hsaputra commented on the issue:

https://github.com/apache/twill/pull/14
  
Congrats @yaojiefeng ! =)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (TWILL-138) Runtime change of Application runnable log level

2016-12-12 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on TWILL-138:
--

Github user hsaputra commented on the issue:

https://github.com/apache/twill/pull/14
  
Congrats @yaojiefeng ! =)


> Runtime change of Application runnable log level
> 
>
> Key: TWILL-138
> URL: https://issues.apache.org/jira/browse/TWILL-138
> Project: Apache Twill
>  Issue Type: New Feature
>  Components: core
>Reporter: Henry Saputra
>Assignee: Yaojie Feng
> Fix For: 0.9.0
>
>
> With TWILL-24 is supported for setting log level when starting the Twill 
> application, next enhancement is to able to send command to update the 
> aggregate log level for a particular runnable in a Twill application.



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


[GitHub] twill pull request #17: Change initial application submission log message fr...

2016-12-12 Thread anwar6953
GitHub user anwar6953 opened a pull request:

https://github.com/apache/twill/pull/17

Change initial application submission log message from DEBUG to INFO.

Should be ok to set this log message to INFO, since it only happens once.
In case there is some issue with the yarn app, having this at INFO helps.
There are later logs at INFO, but if the issue causes the code to not reach 
there, then that isn't sufficient.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/anwar6953/twill master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/twill/pull/17.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #17


commit b890e2fefa8b765c1c6c28c71fdd5cbb72d05620
Author: Ali Anwar 
Date:   2016-12-12T22:48:59Z

Change initial application submission log message from DEBUG to INFO.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] twill issue #17: Change initial application submission log message from DEBU...

2016-12-12 Thread yaojiefeng
Github user yaojiefeng commented on the issue:

https://github.com/apache/twill/pull/17
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] twill issue #17: Change initial application submission log message from DEBU...

2016-12-12 Thread chtyim
Github user chtyim commented on the issue:

https://github.com/apache/twill/pull/17
  
LGTM.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---