Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135423114
@liancheng Thanks for the clear investigation and explanation.
If I understand it correctly, it means that the original direction of this
PR is correct.
---
If
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135392980
One thing to note is that, case sensitivity of Spark SQL is configurable
([see here] [1]). So I don't think we should make `StructType` completely case
insensitive (ye
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135374076
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135374072
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135373958
[Test build #41677 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41677/console)
for PR 7520 at commit
[`a389746`](https://github.
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135334139
Ok. Thanks. Wait for @liancheng's review.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135333184
[Test build #41677 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41677/consoleFull)
for PR 7520 at commit
[`a389746`](https://gith
Github user zhzhan commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135332239
@liancheng have more insights on this part.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135329490
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135329544
Merged build started.
---
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/7520#issuecomment-135329004
@zhzhan @chenghao-intel Thanks for comment. I've updated the PR title and
codes. Please check if it is ok for you.
---
If your project is set up for it, you can reply to
11 matches
Mail list logo