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
 .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
