[GitHub] spark issue #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apache.spa...

2018-10-17 Thread obermeier
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

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2018-10-17 Thread obermeier
GitHub user obermeier reopened a pull request: https://github.com/apache/spark/pull/19408 [SPARK-22180][CORE] Allow IPv6 address in org.apache.spark.util.Utils.parseHostPort External applications like Apache Cassandra are able to deal with IPv6 addresses. Libraries like spark

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2018-10-13 Thread obermeier
Github user obermeier closed the pull request at: https://github.com/apache/spark/pull/19408 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apache.spa...

2018-10-13 Thread obermeier
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] spark issue #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apache.spa...

2018-10-09 Thread obermeier
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] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2017-11-14 Thread obermeier
Github user obermeier commented on a diff in the pull request: https://github.com/apache/spark/pull/19408#discussion_r150909772 --- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala --- @@ -1146,6 +1146,20 @@ class UtilsSuite extends SparkFunSuite

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2017-11-14 Thread obermeier
Github user obermeier commented on a diff in the pull request: https://github.com/apache/spark/pull/19408#discussion_r150907312 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -982,7 +982,19 @@ private[spark] object Utils extends Logging { return

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2017-11-14 Thread obermeier
Github user obermeier commented on a diff in the pull request: https://github.com/apache/spark/pull/19408#discussion_r150798868 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -982,7 +982,19 @@ private[spark] object Utils extends Logging { return

[GitHub] spark issue #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apache.spa...

2017-11-13 Thread obermeier
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

[GitHub] spark issue #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apache.spa...

2017-11-12 Thread obermeier
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: reviews-h

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6 address in org.apa...

2017-10-01 Thread obermeier
Github user obermeier commented on a diff in the pull request: https://github.com/apache/spark/pull/19408#discussion_r142047496 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -981,7 +981,13 @@ private[spark] object Utils extends Logging { return

[GitHub] spark pull request #19408: [SPARK-22180][CORE] Allow IPv6

2017-10-01 Thread obermeier
GitHub user obermeier opened a pull request: https://github.com/apache/spark/pull/19408 [SPARK-22180][CORE] Allow IPv6 External applications like Apache Cassandra are able to deal with IPv6 addresses. Libraries like spark-cassandra-connector combine Apache Cassandra with Apache