Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@jerryshao Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Cannot cleanly merge to 2.2, so this will only land to master branch.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
LGTM, merging to master, if possible to 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81768/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81768 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81768/testReport)**
for PR 19132 at commit
[`06035f7`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81768 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81768/testReport)**
for PR 19132 at commit
[`06035f7`](https://github.com/apache/spark/commit/06
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81764/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81764/testReport)**
for PR 19132 at commit
[`8ec4319`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81764/testReport)**
for PR 19132 at commit
[`8ec4319`](https://github.com/apache/spark/commit/8e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81710/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81710 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81710/testReport)**
for PR 19132 at commit
[`f0e1ded`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81710/testReport)**
for PR 19132 at commit
[`f0e1ded`](https://github.com/apache/spark/commit/f0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81709/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81709/testReport)**
for PR 19132 at commit
[`560d442`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81709/testReport)**
for PR 19132 at commit
[`560d442`](https://github.com/apache/spark/commit/56
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/19132
I've been following, still LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Overall LGTM, @ajbozarth can you please review again?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@jerryshao @HyukjinKwon Jenkins done.Thanks too much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81678/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81678 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81678/testReport)**
for PR 19132 at commit
[`25fe22c`](https://github.com/apache/spark/commit/2
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Thanks @HyukjinKwon , I will ping Josh about this thing ð .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19132
@jerryshao, for triggering tests on Jenkins, I think this should be added
by its admin manually as well if I understood correctly. In my case, I asked
this to Josh Rosen before via email private
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19132
**[Test build #81678 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81678/testReport)**
for PR 19132 at commit
[`25fe22c`](https://github.com/apache/spark/commit/25
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19132
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
Yes i have confirmed.Below is a testing job: @jerryshao
![_001](https://user-images.githubusercontent.com/26762018/30306401-7c682822-97aa-11e7-82ae-9c5802680dd5.png)
![_002](https://user-im
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Looks like I don't have the Jenkins permission to trigger UT ð . Let me
ping @srowen to trigger the test.
---
-
To unsubscr
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@jerryshao Thanks for your time. IIUC, event log is completed since driver
has not dropped any event of executor which has problem described above.See
below,driver only drop two events after shuttin
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Still I have a question about history server, is you event log an
incomplete event log or completed when you met such issue?
---
-
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
Done @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
So your PR description is quite confusing, would you please elaborate your
problem in detail and describe how to reproduce the issue.
---
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
I just appear on history ui since Live ui will use heartbeat metric result
@jerryshao(From my understanding) .It appears when Driver exit unnormally,then
status of taskinfo can not be updated correc
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19132
Is this a problem only in History UI, or it also has issues in Live UI?
From my understanding you only pass a last update time for history UI, so is it
intended?
Also you mentioned "When
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@vanzin @zsxwing could you help reivew this?Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
Ping @kiszk @jerryshao can you help reivew this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@ajbozarth I have updated the implementation which only access FS in
FSHistoryServerProvider
---
-
To unsubscribe, e-mail: revie
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
Thanks for your recommendation @ajbozarth .Could you put a link for your
pr?For the problem you mentioned,i have thought about them.
1ãFsHistoryServer will always use FS to get event log
2ã
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/19132
Though this may fix the problem it is 100% the wrong way to do it for
multiple reasons:
1. The UI should not touch the FS, FS access it abstracted out so the UI is
can work with multiple FS im
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19132
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
44 matches
Mail list logo