Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user obermeier commented on the issue:
https://github.com/apache/spark/pull/19408
If Spark runs in YARN Cluster this issue still exists
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user obermeier commented on the issue:
https://github.com/apache/spark/pull/19408
This issue seems to be fixed in Spark 2.3.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user obermeier commented on the issue:
https://github.com/apache/spark/pull/19408
I total agree with you.
What do you think about just adding a log message if the given string is
obviously not a valid host name.
Because the given _NumberFormatException_ much later after
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19408
Fully agree with @mridulm 's big picture suggestion, and I also think
supporting IPv6 should be designed as a integral feature, instead of just
putting together some PRs.
---
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/19408
To rephrase @jiangxb1987's question - supporting IPv6 is a much larger
effort, which spark currently does not. We should be addressing that as the
problem to solve, and fix this as part of IPv6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85062/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19408
**[Test build #85062 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85062/testReport)**
for PR 19408 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19408
**[Test build #85062 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85062/testReport)**
for PR 19408 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19408
**[Test build #84982 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84982/testReport)**
for PR 19408 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84982/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19408
**[Test build #84982 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84982/testReport)**
for PR 19408 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19408
ok to test
---
-
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/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19408
Sounds good.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user obermeier commented on the issue:
https://github.com/apache/spark/pull/19408
I chose this function because I had some exceptions like this [1] if I used
IPv6 hosts.
In this example ```org.apache.spark.util.Utils$.parseHostPort``` decided to
use f904 as port but it was
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19408
Are you planning to fully address the IPv6 issues? If not, why do we choose
to adapt this single function separately?
---
-
Github user obermeier commented on the issue:
https://github.com/apache/spark/pull/19408
Done
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19408
@obermeier Could you please rebase this with the latest master? Thanks!
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19408
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
29 matches
Mail list logo