Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-177674881
Cherry-picked to branch-1.6.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10388
---
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 user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167642446
Thanks, merging 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167162791
**[Test build #48315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48315/consoleFull)**
for PR 10388 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167162799
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167162798
Merged build finished. Test FAILed.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167180493
**[Test build #48319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48319/consoleFull)**
for PR 10388 at commit
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167179672
I delete the test cases from DataFrameNaFunctionsSuite.scala. I checked the
previous failure, not sure why it is failed. I worked when I run the local test
on my
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167189998
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167189967
**[Test build #48319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48319/consoleFull)**
for PR 10388 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167189997
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167152963
**[Test build #48315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48315/consoleFull)**
for PR 10388 at commit
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48231314
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -194,4 +194,45 @@ class DataFrameNaFunctionsSuite extends
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48230827
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48230905
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48230807
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -194,4 +194,45 @@ class DataFrameNaFunctionsSuite extends
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48230771
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166407905
@marmbrus : Can you help take a look at this PR? Thanks for your review.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166408282
ok to test
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166410817
**[Test build #48127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48127/consoleFull)**
for PR 10388 at commit
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48201572
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48201759
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48199310
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -194,4 +194,45 @@ class DataFrameNaFunctionsSuite extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166436841
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166436837
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-166436691
**[Test build #48127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48127/consoleFull)**
for PR 10388 at commit
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48199539
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -194,4 +194,45 @@ class DataFrameNaFunctionsSuite extends
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48198346
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -88,16 +88,27 @@ private[sql] object
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10388#discussion_r48201852
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -194,4 +194,45 @@ class DataFrameNaFunctionsSuite extends
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-165710284
The failure is because of the changed project, will submit an updated patch
tomorrow.
---
If your project is set up for it, you can reply to this email and have
Github user kevinyu98 closed the pull request at:
https://github.com/apache/spark/pull/10299
---
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 user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-165919581
I will create a new PR.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-165924462
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 user kevinyu98 opened a pull request:
https://github.com/apache/spark/pull/10388
[SPARK-12231][SQL]create a combineFilters' projection when we call
buildPartitionedTableScan
Hello Michael & All:
We have some issues to submit the new codes in the other PR(#10299),
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164604715
**[Test build #47690 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47690/consoleFull)**
for PR 10299 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608983
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608828
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608982
Merged build finished. Test FAILed.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608928
**[Test build #47691 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47691/consoleFull)**
for PR 10299 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608826
Merged build finished. Test FAILed.
---
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 user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164603371
Hello Michael: I fixed the scala style issue, can you help re-run the test?
Thanks.
---
If your project is set up for it, you can reply to this email and have your
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164604210
ok to test
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164605302
**[Test build #47691 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47691/consoleFull)**
for PR 10299 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164608778
**[Test build #47690 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47690/consoleFull)**
for PR 10299 at commit
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10299#discussion_r47566685
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -85,9 +85,11 @@ private[sql] object
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164576672
ok to test
---
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 kevinyu98 opened a pull request:
https://github.com/apache/spark/pull/10299
[SPARK-12231][SQL]create a combineFilters' projection when we call
buildPartitionedTableScan
Hello Michael & All: Here I am submitting another approach to solve this
problem. Can you verify ?
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164578927
**[Test build #47682 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47682/consoleFull)**
for PR 10299 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164579252
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164579242
**[Test build #47682 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47682/consoleFull)**
for PR 10299 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164573774
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10299#issuecomment-164579251
Merged build finished. Test FAILed.
---
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
52 matches
Mail list logo
