[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread blbradley
Github user blbradley commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176321781 I had an unused import. My apologies! --- 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 project

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread blbradley
Github user blbradley commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176338205 @rxin can you start the test again? --- 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 project

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176478692 Thanks - I've merged this. --- 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 project does not have

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176397649 **[Test build #2469 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2469/consoleFull)** for PR 10936 at commit

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread blbradley
Github user blbradley commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176435017 Sweet. Anything else? --- 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 project does not have

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-176434261 **[Test build #2469 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2469/consoleFull)** for PR 10936 at commit

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10936 --- 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 project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-175859804 **[Test build #2467 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2467/consoleFull)** for PR 10936 at commit

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-175860200 **[Test build #2467 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2467/consoleFull)** for PR 10936 at commit

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-27 Thread blbradley
Github user blbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/10936#discussion_r51051135 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/InferSchema.scala --- @@ -135,7 +136,10 @@ private[json] object

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-26 Thread blbradley
GitHub user blbradley opened a pull request: https://github.com/apache/spark/pull/10936 [SPARK-12749][SQL] add json option to parse floating-point types as DecimalType I tried to add this via `USE_BIG_DECIMAL_FOR_FLOATS` option from Jackson with no success. Added test for

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10936#issuecomment-175320473 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: [SPARK-12749][SQL] add json option to parse fl...

2016-01-26 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10936#discussion_r50931235 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/InferSchema.scala --- @@ -135,7 +136,10 @@ private[json] object InferSchema

[GitHub] spark pull request: [SPARK-12749][SQL] add json option to parse fl...

2016-01-26 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10936#discussion_r50931255 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/InferSchema.scala --- @@ -135,7 +136,10 @@ private[json] object InferSchema