You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/04/18 07:33:41 UTC
[jira] [Commented] (FLINK-6317) History server - wrong default
directory
[ https://issues.apache.org/jira/browse/FLINK-6317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15972250#comment-15972250 ]
ASF GitHub Bot commented on FLINK-6317:
---------------------------------------
GitHub user LorenzBuehmann opened a pull request:
https://github.com/apache/flink/pull/3731
[FLINK-6317] Fix wrong default directory for history server
This PR will fix the issue with the missing file separator when using the default Java temp directory for the history server.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/LorenzBuehmann/flink master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/3731.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 #3731
----
commit 85aa6079fe610fe4cce066da90edcc7cf87a9704
Author: Lorenz Buehmann <bu...@informatik.uni-leipzig.de>
Date: 2017-01-06T12:34:13Z
[Flink-5378] Bumped Scopt version to 3.5.0.
This will also allow for using comma-separated values in the CLI.
commit ae62e2d962133e0cb90b26a6d4764be263bdaf41
Author: Lorenz Buehmann <bu...@informatik.uni-leipzig.de>
Date: 2017-04-18T07:15:05Z
Merge branch 'master' of github.com:LorenzBuehmann/flink
commit c815bad3817f22d711c108a97f11c68df2b5d3db
Author: Lorenz Buehmann <bu...@informatik.uni-leipzig.de>
Date: 2017-04-18T07:28:18Z
[FLINK-6317] Fix wrong default directory for history server
----
> History server - wrong default directory
> ----------------------------------------
>
> Key: FLINK-6317
> URL: https://issues.apache.org/jira/browse/FLINK-6317
> Project: Flink
> Issue Type: Bug
> Components: Web Client
> Affects Versions: 1.2.0
> Reporter: Lorenz Bühmann
> Priority: Minor
> Labels: easyfix
>
> When the history server is started without a directory specified in the configuration file, it will use some random directory located in the Java Temp directory. Unfortunately, a file separator is missing:
> {code:title=HistoryServer.java@L139-L143|borderStyle=solid}
> String webDirectory = config.getString(HistoryServerOptions.HISTORY_SERVER_WEB_DIR);
> if (webDirectory == null) {
> webDirectory = System.getProperty("java.io.tmpdir") + "flink-web-history-" + UUID.randomUUID();
> }
> webDir = new File(webDirectory);
> {code}
> It should be
> {code}
> webDirectory = System.getProperty("java.io.tmpdir") + File.separator + "flink-web-history-" + UUID.randomUUID();
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)