Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32382879
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -19,15 +19,27 @@ package org.apache.spark.sql.hive
/**
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6627
---
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/6627#issuecomment-111863871
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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32278001
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -0,0 +1,267 @@
+/*
+ * Licensed to the Apache Software
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32277322
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -0,0 +1,267 @@
+/*
+ * Licensed to the Apache Software
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111303675
I'm okay with this change. A few minor comments otherwise LGTM.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32277894
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -19,15 +19,27 @@ package org.apache.spark.sql.hive
/**
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111303709
Thanks for doing the upgrade BTW!
---
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32282458
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -0,0 +1,267 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32282420
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -0,0 +1,267 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32282547
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -19,15 +19,27 @@ package org.apache.spark.sql.hive
/**
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111336882
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/6627#issuecomment-111336878
[Test build #34735 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34735/console)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111314425
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111314439
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-111314541
[Test build #34735 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34735/consoleFull)
for PR 6627 at commit
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32067595
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -27,7 +27,7 @@ import scala.language.reflectiveCalls
import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32067835
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -27,7 +27,7 @@ import scala.language.reflectiveCalls
import
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110514359
code changes lgtm. I'm a little concerned about the dependency issues, but
I'm not sure what to do about it
---
If your project is set up for it, you can reply to this
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32068495
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -19,15 +19,27 @@ package org.apache.spark.sql.hive
/**
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/6627#discussion_r32068173
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -19,15 +19,27 @@ package org.apache.spark.sql.hive
/**
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110090151
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110090120
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110115280
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110112799
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/6627#issuecomment-110112767
[Test build #34453 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34453/console)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110115296
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110115733
[Test build #34462 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34462/consoleFull)
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110090341
[Test build #34453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34453/consoleFull)
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110152313
[Test build #34462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34462/console)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110152327
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-110072897
I believe the API in 1.0/1.1/1.2 is very similar, if not identical, to
0.14. But this is a problem that we'll have to live with unless Spark is
willing to reimplement
Github user WangTaoTheTonic commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109951833
A high level question: how about the hive 1.0/1.1/1.2? It might be hard to
support so many versions if there's no compatibility between them.
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109190793
[Test build #34252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34252/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109190802
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/6627#issuecomment-109365029
[Test build #34287 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34287/console)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109365049
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109367996
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109367976
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109368504
[Test build #34290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34290/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109360735
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109360751
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109360810
[Test build #34287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34287/consoleFull)
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109424128
[Test build #34307 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34307/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109415356
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109423537
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109423610
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109415345
[Test build #34290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34290/console)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109473533
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/6627#issuecomment-109473502
**[Test build #34307 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34307/console)**
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109060685
[Test build #34215 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34215/consoleFull)
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109166348
[Test build #34252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34252/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109166043
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109165979
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109046340
The tests are not finding some dependencies that don't seem to exist in the
main maven repo anymore but are available in other mirrors. Any suggestions for
what to do
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109090773
org.pentaho#pentaho-aggdesigner-algorithm is still missing after my last
attempt... not sure what's the best way to proceed now (maven central doesn't
have that artifact
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109049828
Or I could remove the tests for 0.14 (and you'd have to trust me that they
pass).
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109073029
[Test build #34215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34215/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109073035
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109058150
(Another minor update: the missing dependencies are all transitive
dependencies of calcite. Not sure if it would be safe to upgrade the calcite
version to a newer one.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109060448
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109060400
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108993676
[Test build #34195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34195/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108993694
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/6627#issuecomment-108973729
[Test build #34195 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34195/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108972303
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108972366
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-109039290
I will let Michael comment on this. Explicitly not @ him so he won't see
this during vacation; otherwise his heart might bleed... :)
---
If your project is set up for
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108645240
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 vanzin opened a pull request:
https://github.com/apache/spark/pull/6627
[SPARK-8065] [hive] Add support for Hive 0.14 metastores
This change has two parts.
The first one gets rid of ReflectionMagic. That worked well for the
differences between 0.12 and
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108658550
[Test build #34133 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34133/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108658558
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108644333
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108653446
[Test build #34136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34136/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108644297
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108648178
[Test build #34133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34133/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108652819
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108652780
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108647722
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108647752
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108645238
[Test build #34132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34132/consoleFull)
for PR 6627 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108669450
[Test build #34136 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34136/consoleFull)
for PR 6627 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6627#issuecomment-108669462
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
83 matches
Mail list logo