Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12520
---
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 ena
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214700491
Merged to master
---
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 featu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214699059
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214699058
Merged build finished. Test PASSed.
---
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214698894
**[Test build #56987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56987/consoleFull)**
for PR 12520 at commit
[`12b5739`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214668768
**[Test build #56987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56987/consoleFull)**
for PR 12520 at commit
[`12b5739`](https://gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214668567
Jenkins retest this please
---
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 user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12520#discussion_r61047232
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -441,9 +441,9 @@ private[sql] object Partitioning
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214642154
**[Test build #2884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2884/consoleFull)**
for PR 12520 at commit
[`12b5739`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214619179
**[Test build #2884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2884/consoleFull)**
for PR 12520 at commit
[`12b5739`](https://g
Github user javawithjiva commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214618915
Thank you for helping me get my first pull request out. Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12520#discussion_r60964326
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -441,7 +441,7 @@ private[sql] object Partitioning
Github user javawithjiva commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214300491
Updated
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-214276411
@javawithjiva if you update this accordingly I'll merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-212953102
Actually, from skimming the code base, I think you've almost got all
instances where valueOf could be parseX. Go for a clean sweep and update the
JIRA. I see the followi
Github user javawithjiva commented on the pull request:
https://github.com/apache/spark/pull/12520#issuecomment-212948656
@srowen or @rxin Can one of you merge?
---
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 proj
16 matches
Mail list logo
