Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Starting Timeline Server in JOBHISTORY to fix job loss issue #503

Merged
merged 12 commits into from
Apr 20, 2018

Conversation

ydye
Copy link
Contributor

@ydye ydye commented Apr 19, 2018

No description provided.

@ydye ydye requested a review from yqwang-ms April 19, 2018 09:51
@coveralls
Copy link

coveralls commented Apr 19, 2018

Coverage Status

Coverage decreased (-0.2%) to 68.501% when pulling 391725f on yuye/time-line-server into 3cc7edf on master.

@yqwang-ms
Copy link
Member

yqwang-ms commented Apr 19, 2018

wait AHS tested ok #Closed

@yqwang-ms
Copy link
Member

http://10.151.40.234:8188/applicationhistory


In reply to: 382681010 [](ancestors = 382681010)

@fanyangCS
Copy link
Contributor

framework launcher unit test failed due to permission denied. @yuqi Wang, can you take a look?


In reply to: 382681351 [](ancestors = 382681351,382681010)

@@ -6,7 +6,7 @@
<!--
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
@@ -14,8 +13,22 @@
@@ -14,8 +13,27 @@
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’m curious about why we need mapred. Could you explain in brief?

@@ -40,6 +40,8 @@ spec:
name: hadoop-config-volume
- mountPath: /host-configuration
name: host-confg-volume
- mountPath: /var/lib/hadoopdata
name: hadoop-tmp-storage
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain what this folder is created for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a path that hadoop will store tmpdata to the path.

Copy link
Member

@yqwang-ms yqwang-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@ydye ydye merged commit 6e97dfc into master Apr 20, 2018
@ydye ydye deleted the yuye/time-line-server branch April 20, 2018 08:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants