Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32891306
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +629,20 @@ class SQLQuerySuite extends QueryTest {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32698499
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +629,20 @@ class SQLQuerySuite extends QueryTest {
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-112994338
Oh, sorry @JoshRosen the meaningless output is caused by #5671, I will fix
it with a followup PR.
---
If your project is set up for it, you can reply to this ema
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-112913378
After this patch, the pull request builder logs show 5 lines of stdout
output, which makes them hard to read:
```
[info] - test script transform for st
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6404
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-111365513
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 have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-111352510
[Test build #34742 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34742/console)
for PR 6404 at commit
[`8418c97`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-111352519
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/6404#issuecomment-111335198
[Test build #34742 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34742/consoleFull)
for PR 6404 at commit
[`8418c97`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-111334917
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-111334862
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 chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32285539
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryT
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32285088
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryT
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32284775
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryTest {
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32284570
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryT
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32284375
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryTest {
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r32284390
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,6 +59,7 @@ case class ScriptTransformation(
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-107871100
LGTM
---
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 chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31502715
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryT
Github user zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31406980
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,6 +59,7 @@ case class ScriptTransformation(
Github user zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31406553
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryTest
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31395732
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryTest {
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31395725
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,6 +59,7 @@ case class ScriptTransformation(
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-107257613
/cc @chenghao-intel
---
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 zhichao-li commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-106647949
cc @rxin @liancheng @marmbrus
---
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 n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105746099
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105746085
[Test build #33554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33554/consoleFull)
for PR 6404 at commit
[`d9677e1`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105746098
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 zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31097497
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31097433
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryTest
Github user zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31097361
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105717778
[Test build #33554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33554/consoleFull)
for PR 6404 at commit
[`d9677e1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105717236
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105717188
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/6404#issuecomment-105466997
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105466998
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105466985
[Test build #33507 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33507/consoleFull)
for PR 6404 at commit
[`c419a64`](https://git
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31016417
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user zhichao-li commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31013646
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31013514
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -629,12 +633,24 @@ class SQLQuerySuite extends QueryT
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31013443
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformati
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31011453
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6404#discussion_r31011382
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala
---
@@ -58,11 +58,12 @@ case class ScriptTransformation(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105429372
[Test build #33507 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33507/consoleFull)
for PR 6404 at commit
[`c419a64`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105428391
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6404#issuecomment-105428362
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 zhichao-li opened a pull request:
https://github.com/apache/spark/pull/6404
[SPARK-7862][SQL]Fix the deadlock in script transformation for stderr
[Related PR SPARK-7044] (https://github.com/apache/spark/pull/5671)
You can merge this pull request into a Git repository by
47 matches
Mail list logo