Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22626
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98405/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22930
**[Test build #98415 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98415/testReport)**
for PR 22930 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98413/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22923
**[Test build #98416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98416/testReport)**
for PR 22923 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22923
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22914
**[Test build #98417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98417/testReport)**
for PR 22914 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544667
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/csvExpressions.scala
---
@@ -174,3 +176,66 @@ case class SchemaOfCsv(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98418/testReport)**
for PR 22913 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
Can we add a test in `ArrowUtilsSuite.scala`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544775
--- Diff: sql/core/src/test/resources/sql-tests/inputs/csv-functions.sql ---
@@ -15,3 +15,10 @@ CREATE TEMPORARY VIEW csvTable(csvField, a) AS SELECT *
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22913#discussion_r230544838
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/arrow/ArrowUtils.scala
---
@@ -71,6 +71,7 @@ object ArrowUtils {
case d:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22504
**[Test build #98413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98413/testReport)**
for PR 22504 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19573
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19573
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544492
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/UnivocityGenerator.scala
---
@@ -15,18 +15,17 @@
* limitations under the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22626
**[Test build #98405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98405/testReport)**
for PR 22626 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22864
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22920
**[Test build #98411 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98411/testReport)**
for PR 22920 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22908#discussion_r230544923
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -56,7 +56,7 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22930
**[Test build #98419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98419/testReport)**
for PR 22930 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22626
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98414/testReport)**
for PR 22921 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22864
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544556
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/CsvFunctionsSuite.scala ---
@@ -45,7 +45,6 @@ class CsvFunctionsSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22914
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22914
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
cc @BryanCutler
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/csvExpressions.scala
---
@@ -174,3 +176,66 @@ case class SchemaOfCsv(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22626#discussion_r230544760
--- Diff: sql/core/src/test/resources/sql-tests/inputs/csv-functions.sql ---
@@ -15,3 +15,10 @@ CREATE TEMPORARY VIEW csvTable(csvField, a) AS SELECT *
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22921
Looks okay to me too but I'd also leave this open for few more days.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98407/
Test PASSed.
---
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22864
Thank you @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22930
**[Test build #98415 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98415/testReport)**
for PR 22930 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98415/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98414/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98414/testReport)**
for PR 22921 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22923
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user itg-abby commented on the issue:
https://github.com/apache/spark/pull/15496
Hi, can you give a ref to the new style guides? I'm not sure if anything
major needs changing.
In the meantime I resolved the one conflict at the bottom of ml/tests.py
and extended the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22504
**[Test build #98407 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98407/testReport)**
for PR 22504 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/22547#discussion_r230528510
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaContinuousInputStream.scala
---
@@ -46,17 +45,22 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22920
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22920
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98411/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22504
**[Test build #98413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98413/testReport)**
for PR 22504 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22847
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22847
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98384/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22482
**[Test build #98387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98387/testReport)**
for PR 22482 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22927
I'll list in as a known issue in 2.4.0, thanks for fixing it!
---
-
To unsubscribe, e-mail:
Github user KyleLi1985 commented on the issue:
https://github.com/apache/spark/pull/22893
> Hm, actually that's the best case. You're exercising the case where the
code path you prefer is fast. And the case where the precision bound applies is
exactly the case where the branch you
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22928
**[Test build #98389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98389/testReport)**
for PR 22928 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user aditanase commented on the issue:
https://github.com/apache/spark/pull/22146
@mccheah agreed on all points, sorry for taking this long to respond. Happy
to see this merged in, can't wait to take it for a spin!
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22482
**[Test build #98385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98385/testReport)**
for PR 22482 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22255
Sure, @gatorsmile .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/22847
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22482
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22847
**[Test build #98386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98386/testReport)**
for PR 22847 at commit
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/22928
[SPARK-25926][CORE] Move config entries in core module to internal.config.
## What changes were proposed in this pull request?
Currently definitions of config entries in `core` module are
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22921#discussion_r230283948
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -279,27 +254,6 @@ class DataFrameSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22928
**[Test build #98388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98388/testReport)**
for PR 22928 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22897
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22897
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98378/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22897
**[Test build #98378 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98378/testReport)**
for PR 22897 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22847
**[Test build #98384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98384/testReport)**
for PR 22847 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22482
**[Test build #98385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98385/testReport)**
for PR 22482 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98385/
Test FAILed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22255
BTW, @rdblue recommended
[key_value_metadata](https://github.com/apache/spark/pull/22255#issuecomment-418169189).
Are we going to `created_by` instead of `key_value_metadata`? Could you give
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22921
LGTM except a minor comment about the test case. Also we need to fix the
PySpark test failure
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22255
That will go like the following.
```
file:
file:/tmp/p/part-7-9dc415fe-7773-49ba-9c59-4c151e16009a-c000.snappy.parquet
creator: parquet-mr version 1.10.0 (build
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22928
cc @jiangxb1987 @cloud-fan @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22878
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22878
**[Test build #98391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98391/testReport)**
for PR 22878 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22878
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user rvesse commented on a diff in the pull request:
https://github.com/apache/spark/pull/22904#discussion_r230318413
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -23,6 +23,18 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98387/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/22530#discussion_r230357101
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -56,10 +61,6 @@ class DefaultSource
with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22928
**[Test build #98388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98388/testReport)**
for PR 22928 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22929
**[Test build #98392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98392/testReport)**
for PR 22929 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22482
**[Test build #98387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98387/testReport)**
for PR 22482 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22929
**[Test build #98393 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98393/testReport)**
for PR 22929 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22530
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22530
**[Test build #98394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98394/testReport)**
for PR 22530 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22530
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22929
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
1 - 100 of 307 matches
Mail list logo