[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-28 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10442#issuecomment-167700718 @microhello please file a JIRA and add it to the title of this PR. See how other patches are opened. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-28 Thread microhello
Github user microhello commented on a diff in the pull request: https://github.com/apache/spark/pull/10442#discussion_r48467911 --- Diff: pom.xml --- @@ -99,14 +99,14 @@ sql/hive unsafe assembly -external/twitter -external/flume -

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-28 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10442#discussion_r48467535 --- Diff: pom.xml --- @@ -99,14 +99,14 @@ sql/hive unsafe assembly -external/twitter -external/flume -

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-28 Thread microhello
Github user microhello commented on the pull request: https://github.com/apache/spark/pull/10442#issuecomment-167738304 @andrewor14 I have created a issue: https://issues.apache.org/jira/browse/SPARK-12548 --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-27 Thread microhello
Github user microhello commented on a diff in the pull request: https://github.com/apache/spark/pull/10442#discussion_r48463834 --- Diff: pom.xml --- @@ -99,14 +99,14 @@ sql/hive unsafe assembly -external/twitter -external/flume -

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-27 Thread microhello
Github user microhello commented on the pull request: https://github.com/apache/spark/pull/10442#issuecomment-167493888 @srowen When I use [facebook/swift](https://github.com/facebook/swift) with spark 1.5.2, throw following exception, so I need add

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10442#issuecomment-166853471 Why does this need to be added? The Guava exceptions are going away in 2.x anyway. This also doesn't follow

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-23 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10442#discussion_r48333932 --- Diff: pom.xml --- @@ -99,14 +99,14 @@ sql/hive unsafe assembly -external/twitter -external/flume -

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10442#issuecomment-166796984 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: Add more exceptions to Guava relocation

2015-12-22 Thread microhello
GitHub user microhello opened a pull request: https://github.com/apache/spark/pull/10442 Add more exceptions to Guava relocation Add `com.google.common.base.AbstractIterator` exceptions to Guava relocation , So far, all guava 14.0.1 `Optional` Class dependencies have added; You