Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/13149#issuecomment-220394198
LGTM
Merging with master and branch-2.0
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63919939
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -706,6 +711,60 @@ class CodegenContext {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63919915
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -740,6 +813,9 @@ abstract class
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63919385
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -717,6 +776,20 @@ abstract class
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r6391
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -740,6 +813,9 @@ abstract class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13182#issuecomment-220392230
**[Test build #58877 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58877/consoleFull)**
for PR 13182 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63918627
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala
---
@@ -124,6 +124,7 @@
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63916719
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -706,6 +711,60 @@ class CodegenContext {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63918110
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala
---
@@ -124,6 +124,7 @@ object
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13163#issuecomment-220390916
**[Test build #58873 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58873/consoleFull)**
for PR 13163 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12850#issuecomment-220390947
**[Test build #58875 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58875/consoleFull)**
for PR 12850 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220390918
**[Test build #58874 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58874/consoleFull)**
for PR 13087 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12313#issuecomment-220390952
**[Test build #58876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58876/consoleFull)**
for PR 12313 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/13027#issuecomment-220390625
@lianhuiwang This PR is useful, other people could easily patch it by
themselves, thanks for it.
---
If your project is set up for it, you can reply to this email and
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/13163#issuecomment-220389680
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
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63916714
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala
---
@@ -124,6 +124,7 @@
Github user rdblue commented on the pull request:
https://github.com/apache/spark/pull/12313#issuecomment-220389967
@cloud-fan, @liancheng, thanks for reviewing! I've rebased on master and
fixed your comments so far.
---
If your project is set up for it, you can reply to this email
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63916876
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -706,6 +711,60 @@ class CodegenContext {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63917072
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -740,6 +813,9 @@ abstract class
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220390039
@sarutak I like this idea, could you simplify it and minimize the changes?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63915953
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -717,6 +776,20 @@ abstract class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13188#discussion_r63915743
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/tpcds/TPCDSQueryBenchmark.scala
---
@@ -0,0 +1,106 @@
+/*
+
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13170#discussion_r63915441
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveCommandSuite.scala
---
@@ -269,6 +270,81 @@ class HiveCommandSuite extends
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63915288
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeFormatter.scala
---
@@ -24,13 +24,13 @@ package
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13170#discussion_r63915128
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -271,6 +274,55 @@ case class LoadData(
}
/**
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13170#discussion_r63915008
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -271,6 +274,55 @@ case class LoadData(
}
/**
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/13189#issuecomment-220385998
@cloud-fan Could you have a screen shot for metrics of BroadcastExchange ?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/13189#discussion_r63914891
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/SQLListener.scala ---
@@ -360,17 +370,27 @@ private[spark] class SQLHistoryListener(conf:
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13170#discussion_r63914894
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -271,6 +274,55 @@ case class LoadData(
}
/**
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/13189#discussion_r63914100
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/BroadcastExchangeExec.scala
---
@@ -66,25 +67,25 @@ case class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10061#discussion_r63914121
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -96,6 +100,7 @@ private[spark] object JsonProtocol {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63913511
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala
---
@@ -124,6 +124,7 @@ object
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10061#discussion_r63913361
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -96,6 +100,7 @@ private[spark] object JsonProtocol {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/13192#discussion_r63913005
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeFormatter.scala
---
@@ -24,13 +24,13 @@ package
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13092#issuecomment-220381427
@l15k4 you can build from branch-2.0.
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63912066
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63912028
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63912009
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13142#issuecomment-220378020
**[Test build #58870 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58870/consoleFull)**
for PR 13142 at commit
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/12951#discussion_r63911572
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/SchedulableBuilder.scala ---
@@ -90,10 +92,16 @@ private[spark] class FairSchedulableBuilder(val
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/12951#issuecomment-220379391
Hi @njwhite, I'm not sure I see a strong need for this -- I posted a msg on
jira (as Kay had earlier). We should keep discussion about the feature in
general there,
Github user kayousterhout commented on the pull request:
https://github.com/apache/spark/pull/12951#issuecomment-220379290
I commented on the JIRA.
---
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/12979#issuecomment-220378999
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/12979#issuecomment-220378989
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/12979#issuecomment-220378688
**[Test build #58871 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58871/consoleFull)**
for PR 12979 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13142#issuecomment-220378345
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/13142#issuecomment-220378344
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/10125#issuecomment-220375820
**[Test build #58872 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58872/consoleFull)**
for PR 10125 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63908615
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63908465
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -348,28 +348,41 @@ case class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220373752
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/10125#issuecomment-220373756
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/10125#issuecomment-220373429
**[Test build #58869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58869/consoleFull)**
for PR 10125 at commit
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13120#discussion_r63903975
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -354,7 +356,27 @@ object
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220361041
I will push the latest one after jenkins finish. Thanks very much !
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10125#discussion_r63900108
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -163,7 +168,9 @@ void validateDataFrameWithBeans(Bean bean,
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10125#discussion_r63900146
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -182,6 +189,8 @@ void validateDataFrameWithBeans(Bean bean,
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220357884
mostly LGTM, pending jenkins.
---
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220357498
For com.fasterxml.jackson.core:jackson-core, I only see 2.7.3.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10125#discussion_r63897785
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -182,6 +189,8 @@ void validateDataFrameWithBeans(Bean bean,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10125#discussion_r63897657
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -163,7 +168,9 @@ void validateDataFrameWithBeans(Bean bean,
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220355804
Looks like paranamer:2.3 is used by avro.
+-org.apache.avro:avro-compiler:1.7.3
| +-commons-lang:commons-lang:2.6
| +-org.apache.avro:avro:1.7.3
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63896316
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -348,28 +348,41 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63896053
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12313#discussion_r63895917
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -498,6 +499,117 @@ class Analyzer(
}
}
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13142#issuecomment-220350310
@rxin yeah. To help code review, I split the code changes to two separate
commits, as shown below
[1) rename two
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220350425
**[Test build #58871 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58871/consoleFull)**
for PR 12979 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13142#issuecomment-220350381
**[Test build #58870 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58870/consoleFull)**
for PR 13142 at commit
Github user techaddict commented on the pull request:
https://github.com/apache/spark/pull/13101#issuecomment-220348713
ping @srowen @andrewor14 @mengxr
---
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 sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/13139#discussion_r63890860
--- Diff: docs/ml-classification-regression.md ---
@@ -374,6 +374,197 @@ regression model and extracting model summary
statistics.
+##
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220345646
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/10125#issuecomment-220347233
**[Test build #58869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58869/consoleFull)**
for PR 10125 at commit
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220346292
retest it 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-220345648
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/13139#discussion_r63890670
--- Diff: docs/ml-classification-regression.md ---
@@ -374,6 +374,197 @@ regression model and extracting model summary
statistics.
+##
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220342987
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/12979#issuecomment-220342991
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/12979#issuecomment-220342761
**[Test build #58867 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58867/consoleFull)**
for PR 12979 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13173#issuecomment-220333996
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/13173#issuecomment-220334004
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/12790#issuecomment-220333799
@BryanCutler Thanks for your clarifying, good point and your proposal makes
sense. The only left issue is whether we can figure out a better way to handle
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13173#issuecomment-220333710
**[Test build #58865 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58865/consoleFull)**
for PR 13173 at commit
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10125#discussion_r63880607
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/DecimalType.scala ---
@@ -109,6 +109,7 @@ object DecimalType extends AbstractDataType {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220321546
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/12979#issuecomment-220321521
**[Test build #58866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58866/consoleFull)**
for PR 12979 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220321548
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/12979#issuecomment-220320973
**[Test build #58867 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58867/consoleFull)**
for PR 12979 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-220319710
**[Test build #58866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58866/consoleFull)**
for PR 12979 at commit
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/13131#issuecomment-220312991
cc @jkbradley
---
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
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/13149#discussion_r63870577
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -206,11 +212,21 @@ object KMeansModel extends MLReadable[KMeansModel] {
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/13194#issuecomment-220311748
cc @jkbradley @mengxr
---
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 yanboliang commented on the pull request:
https://github.com/apache/spark/pull/13195#issuecomment-220311690
cc @MLnick @jkbradley
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13120#discussion_r63869408
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -354,7 +356,27 @@ object
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13173#issuecomment-220310498
**[Test build #58865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58865/consoleFull)**
for PR 13173 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13194#issuecomment-220300668
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/13186#issuecomment-220300727
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/13186#issuecomment-220300728
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/13194#issuecomment-220300669
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/13186#issuecomment-220300558
**[Test build #58859 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58859/consoleFull)**
for PR 13186 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13194#issuecomment-220300587
**[Test build #58864 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58864/consoleFull)**
for PR 13194 at commit
601 - 700 of 811 matches
Mail list logo