Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9935
---
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 enab
Github user maropu closed the pull request at:
https://github.com/apache/spark/pull/10635
---
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 maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-174782722
@marmbrus Okay, I'll do that.
---
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 ha
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-174699659
Okay, this seems good, but can you reopen it against branch-1.6? I don't
think that we want to continue to offer compatibility once the major version
changes in 2.0.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-173579799
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/10635#issuecomment-173579793
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/10635#issuecomment-173579529
**[Test build #49876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49876/consoleFull)**
for PR 10635 at commit
[`57a57fc`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-173556798
**[Test build #49876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49876/consoleFull)**
for PR 10635 at commit
[`57a57fc`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-173553404
@marmbrus ping
---
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 maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171535976
Sorry that yor're confused.
The codes in the link can be compiled because `lib/` has
`/spark-catalyst_2.10-2.0.0-SNAPSHOT.jar` modified by this patch.
So, you nee
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171500570
I'm confused. The repo you link to compiles fine against [Spark
1.6.0](https://github.com/maropu/spark-compat-test/blob/master/build.sbt#L2).
```
> compil
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171499775
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171499779
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171499565
**[Test build #49355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49355/consoleFull)**
for PR 10635 at commit
[`cc971a7`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171482732
**[Test build #49355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49355/consoleFull)**
for PR 10635 at commit
[`cc971a7`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171478073
I found that aggregation functions such as `Max` and `Min` in `
org.apache.spark.sql.catalyst.expressions.aggregate` has the same issue because
the ticket in SPARK-1150
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171471897
@marmbrus @marmbrus checked;
Codes below can be compiled in v1.5.2 though, they cannot be compile in
v.1.6.0.
https://github.com/maropu/spark-compat-test/blob/m
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-171451259
Actually, it looks like this has been taken over in #10635. @smola, do you
mind closing this in favor of the newer active PR? Thanks!
---
If your project is set up fo
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-171451007
Ping @rxin @marmbrus, is this change important for 1.6.1?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-171121913
@marmbrus okay and I'll try it in a day.
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-170122685
I could be missing something, but it seems like since you had to manually
exclude things here to make it compile, users are going to run into problems.
Can you actual
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169866428
@marmbrus @smola ISTM users can compile the source codes that only have
compatibility issues of these types.
---
If your project is set up for it, you can reply to this
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169861548
Does this actually let you use one source to compile against both versions
of Spark?
---
If your project is set up for it, you can reply to this email and have your
r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169861337
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169861338
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169861160
**[Test build #48988 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48988/consoleFull)**
for PR 10635 at commit
[`8bdd481`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169843491
**[Test build #48988 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48988/consoleFull)**
for PR 10635 at commit
[`8bdd481`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169842632
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 this fea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169610530
Test FAILed.
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/10635#issuecomment-169600260
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169600247
**[Test build #48925 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48925/consoleFull)**
for PR 10635 at commit
[`fa45a41`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169598956
**[Test build #48925 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48925/consoleFull)**
for PR 10635 at commit
[`fa45a41`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169586785
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10635#issuecomment-169585945
**[Test build #48914 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48914/consoleFull)**
for PR 10635 at commit
[`c51d95b`](https://gi
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/10635
[SPARK-11780][SQL] Add type aliases backwards compatibility
Rework from #9935 because it's stale.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-160824984
ping. we are getting very close to the first 1.6 RC.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159377597
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159377588
**[Test build #46621 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46621/consoleFull)**
for PR 9935 at commit
[`53011da`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159377601
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159376127
**[Test build #46621 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46621/consoleFull)**
for PR 9935 at commit
[`53011da`](https://gith
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159374337
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
en
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9935#issuecomment-159282503
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 pr
GitHub user smola opened a pull request:
https://github.com/apache/spark/pull/9935
[SPARK-11780][SQL] Add type aliases backwards compatibility.
Added type aliases in org.apache.spark.sql.types for classes
moved to org.apache.spark.sql.catalyst.util.
You can merge this pull reque
43 matches
Mail list logo