Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
Date: Tue, 15 Mar 2016 13:19:33 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@flink.apache.org
Message-ID: <JIRA.12895436.1442926261000.22060.1458047973728@Atlassian.JIRA>
In-Reply-To: <JIRA.12895436.1442926261000@Atlassian.JIRA>
References: <JIRA.12895436.1442926261000@Atlassian.JIRA>
 <JIRA.12895436.1442926261941@arcas>
Subject: [jira] [Commented] (FLINK-2732) Add access to the TaskManagers' log
 file and out file in the web dashboard.
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

ASF GitHub Bot commented on FLINK-2732:
---------------------------------------

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

    https://github.com/apache/flink/pull/1790#discussion_r56160183
  
    --- Diff: flink-runtime/src/main/scala/org/apache/flink/runtime/taskmanager/TaskManager.scala ---
    @@ -781,6 +787,32 @@ class TaskManager(
         }
       }
     
    +  private def handleRequestTaskManagerLog
    +  (sender: ActorRef, requestType: LogTypeRequest, jobManager: ActorRef):
    +  Unit = {
    +    if (blobService.isDefined && jobManager.path.address.host.isDefined) {
    +      //create new Thread to upload log, to not block the TM for too long
    +      new Thread() {
    +        override def run(): Unit = {
    +          val jmHost = jobManager.path.address.host.get
    +          val port = blobService.get.getPort
    +          val logFilePath = System.getProperty("log.file");
    +          val file: File = requestType match {
    +            case LogFileRequest => new File(logFilePath);
    +            case StdOutFileRequest =>
    +              new File(logFilePath.substring(0, logFilePath.length - 4) + ".out");
    +          }
    +
    +          val client: BlobClient = new BlobClient(new InetSocketAddress(jmHost, port))
    +
    +          sender ! client.put(new FileInputStream(file))
    +        }
    +      }.run()
    +    } else {
    +      throw new IOException("No BlobService defined, cannot upload TaskManager logs.")
    --- End diff --
    
    Throwing this exception here will terminate the TM. I think the exception should rather be send to the `sender` where it is logged.


> Add access to the TaskManagers' log file and out file in the web dashboard.
> ---------------------------------------------------------------------------
>
>                 Key: FLINK-2732
>                 URL: https://issues.apache.org/jira/browse/FLINK-2732
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Webfrontend
>    Affects Versions: 0.10.0
>            Reporter: Stephan Ewen
>            Assignee: Chesnay Schepler
>             Fix For: 1.0.0
>
>
> Add access to the TaskManagers' log file and out file in the web dashboard.
> This needs addition on the server side, as the log files need to be transferred   to the JobManager via the blob server.


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