Github user jongyoul commented on the issue:
https://github.com/apache/spark/pull/21213
@gengliangwang Thanks. But changing URI format might make any
incompatibility with other versions, thus we need to consider other aspects
more.
---
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/21213
@jongyoul @ajbozarth
Personally I prefer to the current way, showing the error message:
Github user jongyoul commented on the issue:
https://github.com/apache/spark/pull/21213
Thanks for reviewing this PR. Concerning your comments:
1. It makes sense. I'll prolong the time to wait for the current page.
1. Personally, I wondered if redirecting page would be
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/21213
So I just ran it and have a couple minor issues:
1. The immediate refresh is a bit jarring, maybe put it on a few second
delay (long enough to read the redirect message?)
2. The automatic
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/21213
Code-wsie I think this looks fine, I'll check it out and test it later. But
it seems the Utils file has some old style error that's making the GitHub diff
crazy, mind just fixing it in this pr?
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21213
also cc @gengliangwang
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21213
cc @ajbozarth and @guoxiaolongzte who I believe are interested in this
change.
---
-
To unsubscribe, e-mail:
Github user jongyoul commented on the issue:
https://github.com/apache/spark/pull/21213
Please review this PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90088/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21213
**[Test build #90088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90088/testReport)**
for PR 21213 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2835/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21213
**[Test build #90088 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90088/testReport)**
for PR 21213 at commit
Github user jongyoul commented on the issue:
https://github.com/apache/spark/pull/21213
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90036/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21213
**[Test build #90036 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90036/testReport)**
for PR 21213 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2799/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21213
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jongyoul commented on the issue:
https://github.com/apache/spark/pull/21213
As I described in issue description, yarn filter couldn't pass query
strings when redirecting to yarn application. It's already fixed with the
latest version of yarn, but some old versions still
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21213
**[Test build #90036 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90036/testReport)**
for PR 21213 at commit
22 matches
Mail list logo